Skip to content

Commit

Permalink
Remove vendored first
Browse files Browse the repository at this point in the history
While first is nice to have, it adds a lot of code in vendor.
This patch achieves the same with less code in vendor (~80 lines less).
  • Loading branch information
oz123 committed Jan 9, 2022
1 parent b21baad commit f276360
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 100 deletions.
4 changes: 2 additions & 2 deletions pipenv/core.py
Original file line number Diff line number Diff line change
Expand Up @@ -2525,7 +2525,6 @@ def do_check(
args=None,
pypi_mirror=None
):
from pipenv.vendor.first import first
from pipenv.vendor.vistir.compat import JSONDecodeError

if not system:
Expand Down Expand Up @@ -2569,7 +2568,8 @@ def do_check(
if not system:
python = project._which("python")
else:
python = first(system_which(p) for p in ("python", "python3", "python2"))
interpreters = [system_which(p) for p in ("python", "python3", "python2")]
python = interpreters[0] if interpreters else None
if not python:
click.echo(crayons.red("The Python interpreter can't be found."), err=True)
sys.exit(1)
Expand Down
19 changes: 0 additions & 19 deletions pipenv/vendor/first.LICENSE

This file was deleted.

78 changes: 0 additions & 78 deletions pipenv/vendor/first.py

This file was deleted.

1 change: 0 additions & 1 deletion pipenv/vendor/vendor.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ colorama==0.4.4
distlib==0.3.2
docopt==0.6.2
dparse==0.5.1
first==2.0.2
funcsigs==1.0.2
idna==3.2
importlib-metadata==4.6.1
Expand Down

0 comments on commit f276360

Please sign in to comment.