-
Notifications
You must be signed in to change notification settings - Fork 54
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 duplicated requirements.txt
#798
Conversation
✅ Deploy Preview for olmv1 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
d4b66d5
to
f19017e
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #798 +/- ##
=======================================
Coverage 64.63% 64.63%
=======================================
Files 16 16
Lines 1315 1315
=======================================
Hits 850 850
Misses 403 403
Partials 62 62
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: Mikalai Radchuk <[email protected]>
Rebased to resolve conflict in |
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 nice one!
Description
We had two
requirements.txt
: one for local builds and one for Netlify. This PR removes one of them and makes both setups use the same file.Reviewer Checklist