eclipse.jdt.ls icon indicating copy to clipboard operation
eclipse.jdt.ls copied to clipboard

make use of GitHub functions for library update and security analysis

Open carstenartur opened this issue 2 years ago • 5 comments

Is it possible to make use of some standard functions to check dependency updates and security issues?

carstenartur avatar Jun 25 '22 17:06 carstenartur

Can one of the admins verify this patch?

eclipse-ls-bot avatar Jun 25 '22 17:06 eclipse-ls-bot

The codeql-analysis seems nearly identical to what we do in https://github.com/eclipse/lemminx/blob/master/.github/workflows/codeql-analysis.yml , so should be fine. What does the dependabot config provide that's different from https://github.com/eclipse/eclipse.jdt.ls/issues?q=dependabot that I assume is configured on the repository ? CC'ing @fbricon in case he may know.

rgrunber avatar Jun 27 '22 20:06 rgrunber

The codeql-analysis seems nearly identical to what we do in https://github.com/eclipse/lemminx/blob/master/.github/workflows/codeql-analysis.yml , so should be fine. What does the dependabot config provide that's different from https://github.com/eclipse/eclipse.jdt.ls/issues?q=dependabot that I assume is configured on the repository ? CC'ing @fbricon in case he may know.

After I activated it in my fork the pull requests for update of external dependencies have been created automatically. Many dependencies are outdated so I assumed it was not or not completely configured.

See https://github.com/carstenartur/eclipse.jdt.ls

carstenartur avatar Jun 27 '22 21:06 carstenartur

@fbricon , I don't see the repo-level settings for JDT-LS. Do we have some dependabot app running there currently that does some very specific dependency updates ? We did in the past, so I wouldn't mind restoring this through GH actions.

rgrunber avatar Jun 28 '22 18:06 rgrunber

Looks pretty good to me. However, I noticed the following PRs when I tried this against my local fork : https://github.com/rgrunber/eclipse.jdt.ls/pulls . Is there any way to exclude a folder from the computation ? Last I checked it was only possible to exclude artifacts by GAV. If that's our only option currently, I think we should just exclude the junit artifact since we aren't depending on it directly.

Others have disabled the feature due to noise from the testing folders.

rgrunber avatar Aug 02 '22 15:08 rgrunber

Not sure how I managed to close this. I'll re-create and merge now.

rgrunber avatar Sep 22 '22 18:09 rgrunber