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

ci: adds codeql-analysis.yml for codeql execution #650

Merged
merged 3 commits into from Jun 28, 2022
Merged

Conversation

nickfloyd
Copy link
Contributor

This enables codeql for this repo to meet the security and stability requirements of the octokit org.

@ghost ghost added this to Inbox in JS Jun 28, 2022
@nickfloyd nickfloyd added the Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR label Jun 28, 2022
@ghost ghost moved this from Inbox to Maintenance in JS Jun 28, 2022
@wolfy1339
Copy link
Member

The CodeQL warning that shows up is not relevant. The HTML is purely for caching purposes, it is never served.

As for bb08b29, can you please revert it. We're working on it in #641

@nickfloyd
Copy link
Contributor Author

The CodeQL warning that shows up is not relevant. The HTML is purely for caching purposes, it is never served.

As for bb08b29, can you please revert it. We're working on it in #641

Absolutely. Now that you mention it, I remember this coming up before. Sorry for the noise here. I'll get it cleaned up.

@nickfloyd nickfloyd requested a review from wolfy1339 June 28, 2022 17:06
@nickfloyd nickfloyd merged commit f116747 into master Jun 28, 2022
@nickfloyd nickfloyd deleted the enables-codeql branch June 28, 2022 17:46
JS automation moved this from Maintenance to Done Jun 28, 2022
@octokitbot
Copy link
Collaborator

🎉 This PR is included in version 6.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR
Projects
No open projects
JS
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants