8000 feat(actions): add codeql code scanning by DioEgizio · Pull Request #1177 · PolyMC/PolyMC · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat(actions): add codeql code scanning #1177

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Oct 8, 2022
Merged

feat(actions): add codeql code scanning #1177

merged 3 commits into from
Oct 8, 2022

Conversation

DioEgizio
Copy link
Contributor

This pr adds code scanning with codeql, to replace the old LGTM.com which is gonna die later this year.

@ryanccn
Copy link
Contributor
ryanccn commented Oct 2, 2022

LGTM.com doesn't LGTM

@DioEgizio
Copy link
Contributor Author

https://github.blog/2022-08-15-the-next-step-for-lgtm-com-github-code-scanning/

@TayouVR
Copy link
Contributor
TayouVR commented Oct 2, 2022

For those repositories, we will create pull requests that add a GitHub Actions workflow that runs code scanning

won't github already make a PR for this themselves?

@DioEgizio
Copy link
Contributor Author

For those repositories, we will create pull requests that add a GitHub Actions workflow that runs code scanning

won't github already make a PR for this themselves?

I don't know but I did it myself anyways

@DioEgizio
feat(actions): add codeql code scanning
4127640 
Signed-off-by: DioEgizio <83089242+DioEgizio@users.noreply.github.com>
@DioEgizio
feat(code scanning): enable security-and-quality query
23b3990 
Signed-off-by: DioEgizio <83089242+DioEgizio@users.noreply.github.com>
@DioEgizio
fix: remove old unused lgtm.yml, exclude cpp/fixme-comment
8467
80e9eed 
Signed-off-by: DioEgizio <83089242+DioEgizio@users.noreply.github.com>
@flowln flowln merged commit e436f47 into PolyMC:develop Oct 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@flowln flowln flowln approved these changes

@Scrumplex Scrumplex Scrumplex approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@DioEgizio @ryanccn @TayouVR @flowln @Scrumplex
0