-
Notifications
You must be signed in to change notification settings - Fork 327
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bump minimum CodeQL bundle version to 2.6.3 #1358
Conversation
Our docs for direct CodeQL CLI use recommend CodeQL 2.6.3 even for GHES 3.1 and 3.2, so this bump is acceptable even if we have a few customers gradually upgrading from deprecated GHES versions. |
c3f3b6e
to
2da27c1
Compare
We now only need to support doing this in the CLI.
It is 2.8.2, not 2.7.3.
e191774
to
0c3e493
Compare
All checks have passed. The PR isn't green yet since it updates the checks to use newer bundle versions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change looks good. Nice to get rid of some code. Looks like there are some merge conflicts now, though.
Resolved conflicts by rerunning the workflow generator and rebuilding the Action. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#dead-code-society
Now that GHES 3.2 is deprecated, we can clean up a bunch of tech debt in the Action by bumping our minimum CodeQL Bundle version to 2.6.3, the first version that went into GHES 3.3.
The highlight here is that the Action no longer needs to count lines of code itself, since now all supported versions of the CLI can do that. This allows us to remove a bunch of code and 14 dependencies!
Merge / deployment checklist