Dependencies#50
Conversation
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 0 |
| Duplication | 0 |
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## dev #50 +/- ##
=======================================
Coverage 98.73% 98.73%
=======================================
Files 9 9
Lines 235 235
Branches 26 26
=======================================
Hits 232 232
Misses 2 2
Partials 1 1 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
alirezazolanvari
left a comment
There was a problem hiding this comment.
Thanks for the efforts. I just asked a question. Other than that, everything else looks fine to me.
| from distutils.core import setup | ||
|
|
||
|
|
||
| def get_requires() -> list: |
There was a problem hiding this comment.
I’m not sure why this function was removed. Could you please explain why this change is necessary? Without it, adding each dependency is going to be a hassle since it needs to be updated in several places.
Reference Issues/PRs
What does this implement/fix? Explain your changes.
Any other comments?