-
-
Notifications
You must be signed in to change notification settings - Fork 751
Better --install-all-deps #2730
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
base: dev
Are you sure you want to change the base?
Conversation
📊 Performance Benchmark Report
📈 Detailed Results (All Benchmarks)
🎯 Performance Summary✅ No significant performance changes detected (all changes <10%) 🐍 Python Version 3.11.14 |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev #2730 +/- ##
=====================================
+ Coverage 92% 92% +1%
=====================================
Files 411 410 -1
Lines 33951 33875 -76
=====================================
- Hits 30959 30900 -59
+ Misses 2992 2975 -17 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
This PR improves the
--install-all-depsto also install wordlists etc. in addition to code dependencies. This was accomplished by separating those parts ofsetup()into a newsetup_deps().It also publishes a new dockerfile that has all module dependencies preinstalled.