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 codeql-analysis.yml #3975

Closed
wants to merge 2 commits into from
Closed

Update codeql-analysis.yml #3975

wants to merge 2 commits into from

Conversation

JJediny
Copy link
Member

@JJediny JJediny commented Sep 20, 2024

CodeQL Action v2 will be deprecated on December 5th, 2024. Please update all occurrences of the CodeQL Action in your workflow files to v3. For more information, see https://github.blog/changelog/2024-01-12-code-scanning-deprecation-of-codeql-action-v2/

Changes proposed in this pull request:

Fix broken codeql action

security considerations

[Note the any security considerations here, or make note of why there are none]

CodeQL Action v2 will be deprecated on December 5th, 2024. Please update all occurrences of the CodeQL Action in your workflow files to v3. For more information, see https://github.blog/changelog/2024-01-12-code-scanning-deprecation-of-codeql-action-v2/
@JJediny JJediny requested a review from a team as a code owner September 20, 2024 17:26
@mgwalker
Copy link
Member

mgwalker commented Oct 2, 2024

Sorry for being so late to this. Since we have enabled code scanning in the repo settings, it looks like we need to remove the CodeQL workflow from the repo, as you can no longer have both:

github/codeql-action#1558

If that makes sense to you too, I've created #3978 to do that.

@mgwalker
Copy link
Member

mgwalker commented Oct 3, 2024

Ah dang, I completely forgot this was open and I merged the other one. 🤦🏻. @JJediny If we need to put the local CodeQL config back in place, please ping me and I'll take care of it since it's my fault. 😬

@JJediny
Copy link
Member Author

JJediny commented Oct 4, 2024 via email

@JJediny JJediny closed this Oct 4, 2024
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