Skip to content

Conversation

@ncdc
Copy link
Member

@ncdc ncdc commented Jul 28, 2023

Description

Enable receiving merge_group events in preparation for enabling the merge queue.

Reviewer Checklist

  • API Go Documentation
  • Tests: Unit Tests (and E2E Tests, if appropriate)
  • Comprehensive Commit Messages
  • Links to related GitHub Issue(s)

Signed-off-by: Andy Goldstein <[email protected]>
@ncdc ncdc requested a review from a team as a code owner July 28, 2023 16:54
@codecov
Copy link

codecov bot commented Jul 28, 2023

Codecov Report

Merging #305 (14e54ee) into main (14c9a3d) will decrease coverage by 0.25%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #305      +/-   ##
==========================================
- Coverage   80.69%   80.44%   -0.25%     
==========================================
  Files          18       18              
  Lines         803      803              
==========================================
- Hits          648      646       -2     
- Misses        112      113       +1     
- Partials       43       44       +1     
Flag Coverage Δ
e2e 55.54% <ø> (-0.25%) ⬇️
unit 77.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1 file with indirect coverage changes

Copy link
Contributor

@tmshort tmshort left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jul 28, 2023
@ncdc ncdc merged commit 0be95ca into operator-framework:main Jul 28, 2023
@ncdc ncdc deleted the enable-merge-queue branch August 4, 2023 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants