Skip to content
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

Update to CodeQL version 2.14.4 #82

Merged
merged 3 commits into from
Oct 11, 2023

Conversation

NateD-MSFT
Copy link
Contributor

@NateD-MSFT NateD-MSFT commented Oct 5, 2023

This PR updates the base CodeQL version to 2.14.4 in anticipation of new DataFlow analysis in the future and to stay up to date with current bugfixes, library changes, and the like.

Checklist for Pull Requests

  • Description is filled out.
  • [N/A] Only one query or related query group is in this pull request.
  • [N/A] The version number on changed queries has been increased via the @version comment in the file header.
  • All unit tests have been run: (Test README.md).
  • Commands codeql database create and codeql database analyze have completed successfully.
  • [N/A] A .qhelp file has been added for any new queries or updated if changes have been made to an existing query.

Update cpp-all to 0.9.2, cpp-queries to 0.7.4
Signed-off-by: NateD-MSFT <34494373+NateD-MSFT@users.noreply.github.com>
@Cymoki
Copy link
Member

Cymoki commented Oct 5, 2023

Nate,
What was the outcome of validation for these changes?

@NateD-MSFT
Copy link
Contributor Author

NateD-MSFT commented Oct 5, 2023

Nate, What was the outcome of validation for these changes?

@Cymoki Our unit tests showed no diff compared to running with the old CLI.

@NateD-MSFT NateD-MSFT merged commit b1cc585 into development Oct 11, 2023
2 checks passed
@NateD-MSFT NateD-MSFT deleted the nated-msft/codeql-2144-update branch October 11, 2023 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants