-
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
Handle multiple extras and invalid extras #103
Conversation
944b73a
to
43c9440
Compare
src/poetry_plugin_export/command.py
Outdated
# Checking extras | ||
extras = [] | ||
for extra in self.option("extras"): | ||
if " " in extra: |
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.
I realise I pointed you at code like this to copy, but it occurs to me now that the two branches are unnecessary, this first branch works fine for both cases.
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.
in fact come to think of it isn't this whole thing just
extras += extra.split()
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.
Not quite, I think it would be:
extras.extend(extra.split())
Anyway I changed it to a double comprehension, 🤞 I don't get sent straight to hell
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.
I think my formulation was fine and I like it a little better, but it's not worth arguing over!
ce50916
to
a6cb5ab
Compare
Kudos, SonarCloud Quality Gate passed! |
resolves: #31
resolves: #33