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

Revert "Merge pull request #17938 from MathiasVP/fix-fp-in-missing-check-scanf-fixing-take-2" #18057

Merged
merged 1 commit into from
Nov 21, 2024

Conversation

jketema
Copy link
Contributor

@jketema jketema commented Nov 21, 2024

This reverts commit 6785b93, reversing changes made to d9b86f5.

…ing-check-scanf-fixing-take-2"

This reverts commit 6785b93, reversing
changes made to d9b86f5.
Copy link
Contributor

@MathiasVP MathiasVP left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The revert LGTM, but I'll let @alexet handle the actual approve to not interfere too much with the release process 😄

@alexet
Copy link
Contributor

alexet commented Nov 21, 2024

I think it might be better if I do the step reverting the release prep first. That will conflict with this though as it will revert creating the changelog.

@jketema
Copy link
Contributor Author

jketema commented Nov 21, 2024

I think it might be better if I do the step reverting the release prep first. That will conflict with this though as it will revert creating the changelog.

Happy with that approach too. I just don't want to end up in a situation where we drag in all kinds of new changes.

@alexet alexet merged commit ed922f6 into github:codeql-cli-2.19.4 Nov 21, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants