🌱 Update Tilt to use a single manifest #2082
Conversation
This makes tilt use the equivalent of the experimental manifest with the additional patches. Git seems to get confused about the rename because the two patch files are identical. Signed-off-by: Todd Short <tshort@redhat.com>
✅ Deploy Preview for olmv1 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: joelanford 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 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2082 +/- ##
==========================================
+ Coverage 73.33% 73.43% +0.09%
==========================================
Files 77 77
Lines 7076 7076
==========================================
+ Hits 5189 5196 +7
+ Misses 1545 1540 -5
+ Partials 342 340 -2
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:
|
|
The codecov website is showing 73.43% coverage... so not sure why it's reporting poorly. |
This makes tilt use the equivalent of the experimental manifest with the additional patches.
Git seems to get confused about the rename because the two patch files are identical.
Description
Reviewer Checklist