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

Added Pinned pkgs info #103

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

EaglePPP
Copy link

e.g:
pywin32==306 is available (you have 304) [Pinned]
tornado==6.3.2 is available (you have 6.1)

Tested only on Windows

e.g:
pywin32==306 is available (you have 304) [Pinned]
tornado==6.3.2 is available (you have 6.1)
Tested only on Windows
Copy link
Owner

@jgonggrijp jgonggrijp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great suggestion, thanks!

I want to merge this, but I have three requests:

  • Please add a test so that the feature can be verified in CI. I think this feature is doable with the existing cram test setup.
  • Please use the word "constrained" instead of "pinned". This makes it less ambiguous with "pinned dependencies" in a requirements.txt.
  • Please make it clear in the output which version is listed in the constraints file. If it is guaranteed to be equal to the installed version, I suggest pywin32==306 is available (you have 304, constrained). Otherwise, I suggest pywin32==306 is available (you have 304) [Constrained to 304].

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.

2 participants