-
Notifications
You must be signed in to change notification settings - Fork 329
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
Allow omitting the autobuild Action when build mode specified #2120
Allow omitting the autobuild Action when build mode specified #2120
Conversation
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.
Cool! I like how simple that test was 💖
3bd00e8
to
f3ced61
Compare
@angelapwen I added a couple more tests for the other build modes. Would you mind taking another look? |
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.
The new tests look good to me! Should this have a changelog note? Or will we wait for a general larger-scale announcement of the new build modes?
Right now, this is just for internal testing. Once we've finished testing, we can remove all the internal labels and add a changelog note with all the changes. |
This PR enables users to omit the
autobuild
Action when the build mode is specified and the CLI supports thetraceCommandUseBuildMode
feature (available in CodeQL CLI 2.16.2 and later). This will remove redundancy and reduce the possibility for errors.Merge / deployment checklist