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 #736

Merged
merged 3 commits into from
Feb 8, 2025
Merged

Update codeql-analysis.yml #736

merged 3 commits into from
Feb 8, 2025

Conversation

willem-bd
Copy link
Contributor

Updated the go-lang version to fix the error of codeql.

Updated the go-lang version to fix the error of codeql.
Upgrade the codeql-action version to v3 as action running information suggested.
Upgrade the cache-action version to v3 as action running information suggested.
@willem-bd willem-bd requested a review from AsterDY February 8, 2025 01:59
@willem-bd
Copy link
Contributor Author

As we already enable the codeql scan default from GitHub org, another solution is to remove the codeql-analysis.yml if we don't have special settings on it, we can leverage the GitHub services to handle this kind of problems.

@GuangmingLuo
Copy link
Member

I suggest removing this file

Remove the file to leverage the GitHub default codeql setting to the code scanning.
@liuq19 liuq19 merged commit d8ef674 into main Feb 8, 2025
50 of 51 checks passed
@liuq19 liuq19 deleted the codeql-fix branch February 8, 2025 05:33
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.

3 participants