Skip to content
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

Ignore editable requirements when parsing the req file #75

Merged
merged 1 commit into from
Jul 13, 2021

Conversation

bagerard
Copy link
Contributor

Fixes #74
Relates to #33

I suggest to ignore editable requirements, I don't see good reasons to not to so but if you want me to do things differently, just let me know. It is also possible to report the ignored packages on stdout if you believe that is useful.

@ochedru
Copy link
Collaborator

ochedru commented Jul 13, 2021

Thank you for contributing! This indeed looks like a reasonable solution.

@ochedru ochedru merged commit 5727329 into dhatim:master Jul 13, 2021
@bagerard
Copy link
Contributor Author

Great, thanks for the quick feedback !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Editable requirements (-e) raises "DistributionNotFound: The 'None' distribution was not found"
3 participants