reynoldsnlp / pipster

pipster: The pythonic way to `pip install`.
MIT License
8 stars 2 forks source link

inform about bypassing pipfile #2

Closed reynoldsnlp closed 5 years ago

reynoldsnlp commented 5 years ago

From @ncoghlan here

If Pipfile or Pipfile.lock is seen, emit a warning to stderr about it being bypassed (such a warning could potentially be added to pip by default regardless of how it's invoked)

ncoghlan commented 5 years ago

Hmm, this one is actually a bit more complicated than I originally thought.

  1. You don't want to warn about this at import time, you only want to emit a warning when actually performing the installation
  2. To figure out which Pipfile entries (if any) are relevant is going to require that the warning code be at least somewhat aware of the meaning of the pip CLI arguments, where Pipfile entries would be found relative to those locations, and where installations would end up based on the currently active virtual environment (if any)

Given the flexibility that virtual environments offer in separating the current working directory from the virtual environment site-packages and the location of Pipfile, it may be better to defer this feature for now.