-
Notifications
You must be signed in to change notification settings - Fork 151
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: Remove unused yq test matchers package #1478
Conversation
/lgtm |
/test opendatahub-operator-e2e |
/retest |
/retest |
@grdryn you may need to rebase to get the propagation policy fix |
I think Prow tests the merge of the source branch into the target branch, rather than just the tip of the source branch. If it fails then I'll rebase 🙂 |
Ah ok, just saw GitHub notification: |
Huh, seems like it's not 🤔 I wonder if I have the wrong understanding in general or if we've got it configured differently for this project. Or maybe it does that merge test before merging only if it determines that the target has changed compared to what it previously tested against. In any case, I'll rebase now 🙂 |
Since this appears to be unused, it seems like a good idea to remove it, since that causes a bunch of dependencies to be removed from the go.mod file.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1478 +/- ##
=======================================
Coverage ? 18.72%
=======================================
Files ? 154
Lines ? 10276
Branches ? 0
=======================================
Hits ? 1924
Misses ? 8139
Partials ? 213 ☔ View full report in Codecov by Sentry. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: lburgazzoli The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Description
Since this appears to be unused, it seems like a good idea to remove
it, since that causes a bunch of dependencies to be removed from the
go.mod file.
How Has This Been Tested?
n/a
Screenshot or short clip
Merge criteria