-
Notifications
You must be signed in to change notification settings - Fork 83
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 requirements-parser requirement #3978
Conversation
gsheni
commented
Feb 2, 2023
•
edited
Loading
edited
- Fixes Remove requirements parser requirement #3979
- The requirements-parser is not being updated on conda-forge and is a few versions behind the one on pypi
Codecov Report
@@ Coverage Diff @@
## main #3978 +/- ##
=======================================
- Coverage 99.7% 99.7% -0.0%
=======================================
Files 347 347
Lines 36892 36896 +4
=======================================
+ Hits 36772 36775 +3
- Misses 120 121 +1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
all_specs = ",".join(["".join(spec) for spec in package.specs]) | ||
name = package.name | ||
if convert_to_conda and name in CONDA_TO_PIP_NAME: | ||
name = CONDA_TO_PIP_NAME.get(package.name) |
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.
Maybe not a big deal, but should we add a test (or parameterize the existing test if we can) to hit this line?
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 other than adding the missing coverage