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

chore: delete unneeded custom codeql GitHub Action #276

Merged
merged 1 commit into from
May 8, 2024

Conversation

jmeridth
Copy link
Member

@jmeridth jmeridth commented May 8, 2024

Pull Request

Proposed Changes

We do not have any custom CodeQL jobs, so we are going back to the default CodeQL setup.

New default CodeQL setup is already running successfully

Readiness Checklist

Author/Contributor

  • If documentation is needed for this change, has that been included in this pull request
  • run make lint and fix any issues that you have introduced
  • run make test and ensure you have test coverage for the lines you are introducing

Reviewer

  • Label as either fix, documentation, enhancement, infrastructure, maintenance, or breaking

We do not have any custom CodeQL jobs, so we are going back to the default CodeQL setup.

[New default CodeQL setup is already running successfully](https://github.com/github/issue-metrics/actions/workflows/github-code-scanning/codeql)
@jmeridth jmeridth self-assigned this May 8, 2024
@jmeridth jmeridth requested a review from zkoppert as a code owner May 8, 2024 18:01
@jmeridth jmeridth merged commit 1c3ce5f into main May 8, 2024
27 checks passed
@jmeridth jmeridth deleted the jm-remove-custom-codeql-action branch May 8, 2024 18:07
jmeridth added a commit that referenced this pull request May 28, 2024
- [x] auto-labeler action
- [x] release action
- [x] fix codeql bade in README (after we removed custom workflow in #276)

Signed-off-by: jmeridth <jmeridth@gmail.com>
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.

1 participant