-
-
Notifications
You must be signed in to change notification settings - Fork 611
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
Enhance the option lines in output #448
Enhance the option lines in output #448
Conversation
81e67bc
to
896264d
Compare
Mark the merged PRs of pip-tools in Change Log more clearly so that it is easier to keep track which fixes/features are already in. This includes following 7 PRs: * jazzband#355 * jazzband#378 * jazzband#389 * jazzband#417 * jazzband#441 * jazzband#448 * jazzband#450
piptools/scripts/compile.py
Outdated
@@ -46,6 +46,8 @@ class PipCommand(pip.basecommand.Command): | |||
help="Add header to generated file") | |||
@click.option('--index/--no-index', is_flag=True, default=True, | |||
help="Add index URL to generated file") | |||
@click.option('--emit-trusted-host/--no-trusted-host', is_flag=True, default=True, |
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.
To avoid confusion, the --no flag should provide symmetry to the positive version: --no-emit-trusted-host.
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.
agree and done
…p-tools Merge latest changes from jazzband#448 PR. Also updated the Change Log entry. * 'feature/enhance-option-lines' of github.com:tonyseek/pip-tools: Rename "no-trusted-host" to "no-emit-trusted-host"
Since things have changed a bit, think you could rebase? I'd love to get this in, inconsistent |
piptools/utils.py
Outdated
for x in iterable: | ||
if x not in emitted: | ||
yield x | ||
emitted.add(x) |
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.
The merged PR #468, which does a similar fix for index-urls unicity, uses the OrderedDict idiom to get unique entries, could you reuse that?
d7ed6d5
to
c12d005
Compare
@Groxx @davidovich Thanks for your review. The branch is up-to-date now and the revision is pushed. |
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.
Thanks @tonyseek!
Wondering if removing --extra-index-url from requirements.txt output is possible as well. |
Description
There is a proposal to fix following output:
Those output may be caused by multiple option sources (e.g.
env PIP_INDEX_URL=$U PIP_EXTRA_INDEX_URL=$U pip-compile --extra-index-url=$U x.in
).Changes
--no-trusted-host
: Don't add trusted host to generated file #382