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

(docs) Remove KCP reference #325

Merged
merged 1 commit into from
Aug 22, 2023
Merged

Conversation

anik120
Copy link
Contributor

@anik120 anik120 commented Aug 7, 2023

Description

Reviewer Checklist

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

@anik120 anik120 requested a review from a team as a code owner August 7, 2023 18:50
@codecov
Copy link

codecov bot commented Aug 7, 2023

Codecov Report

Merging #325 (5e93cf7) into main (949c06a) will increase coverage by 1.43%.
Report is 8 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #325      +/-   ##
==========================================
+ Coverage   80.69%   82.13%   +1.43%     
==========================================
  Files          18       21       +3     
  Lines         803      890      +87     
==========================================
+ Hits          648      731      +83     
+ Misses        112      110       -2     
- Partials       43       49       +6     
Flag Coverage Δ
e2e 62.35% <ø> (+6.56%) ⬆️
unit 78.10% <ø> (+1.10%) ⬆️

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

see 9 files with indirect coverage changes

@kevinrizza
Copy link
Member

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Aug 7, 2023
Signed-off-by: Anik <anikbhattacharya93@gmail.com>
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Aug 8, 2023
Copy link
Contributor

@everettraven everettraven 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 Aug 14, 2023
@ncdc ncdc added this pull request to the merge queue Aug 22, 2023
Merged via the queue into operator-framework:main with commit 6770469 Aug 22, 2023
9 checks passed
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.

None yet

4 participants