-
Notifications
You must be signed in to change notification settings - Fork 71
🌱 Remove unfinished uninstall e2e test #2452
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
🌱 Remove unfinished uninstall e2e test #2452
Conversation
Signed-off-by: Per Goncalves da Silva <pegoncal@redhat.com>
✅ Deploy Preview for olmv1 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR removes an unfinished e2e test file (uninstall.feature) that was accidentally included in a previous PR. The file contained a feature test for uninstalling ClusterExtensions but was not yet complete or intended for merge.
Changes:
- Remove the incomplete
uninstall.featurefile from the e2e test suite
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
camilamacedo86
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: camilamacedo86 The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2452 +/- ##
==========================================
- Coverage 73.75% 69.60% -4.15%
==========================================
Files 101 101
Lines 7768 7754 -14
==========================================
- Hits 5729 5397 -332
- Misses 1589 1925 +336
+ Partials 450 432 -18
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
/override codecov/project |
|
@perdasilva: Overrode contexts on behalf of perdasilva: codecov/project DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
8167ff8
into
operator-framework:main
Description
An unrelated file sneaked in to a previous PR 😮💨
Reviewer Checklist