Skip to content
This repository was archived by the owner on May 1, 2025. It is now read-only.

Conversation

@jmeridth
Copy link
Member

Pull Request

Proposed Changes

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 bug, documentation, enhancement, infrastructure, maintenance or breaking

- [x] auto-labeler action
- [x] release action
- [x] fix codeql bade in README (after we removed custom workflow in #70)

Signed-off-by: jmeridth <jmeridth@gmail.com>
@jmeridth jmeridth self-assigned this May 28, 2024
@zkoppert
Copy link
Member

Preview still shows codeql badge broken. Does it need to be merged to work?

@jmeridth jmeridth merged commit 5d3856b into main May 28, 2024
@jmeridth jmeridth deleted the jm_pull_request_target_only branch May 28, 2024 19:01
@jmeridth
Copy link
Member Author

Preview still shows codeql badge broken. Does it need to be merged to work?

yep, unfortunately. Merged and confirmed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants