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.

Sign up for GitHub

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

WIP: Make install editable option an option #2299

Closed
wants to merge 13 commits into from
Closed

WIP: Make install editable option an option #2299

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
15d5fa1
Make install editable option an option
mlouielu Jun 3, 2018
cbf398f
Fix editable
mlouielu Jun 28, 2018
5a43199
Merge branch 'master' into make-install-editable-option
mlouielu Jun 28, 2018
1997e58
Fix test error string and add --editable test
mlouielu Jun 28, 2018
5a13e3e
Fix typo
mlouielu Jun 28, 2018
ab47386
Merge branch 'master' into make-install-editable-option
mlouielu Jun 29, 2018
5a9c3b0
Merge branch 'master' of https://github.com/pypa/pipenv into make-ins…
mlouielu Jul 4, 2018
d0c2369
Fix the single quote in cli option
mlouielu Jul 4, 2018
adebd4b
Add multiple to editable option
mlouielu Jul 4, 2018
15f3a95
Add multiple editable test case
mlouielu Jul 4, 2018
b363ade
Merge branch 'make-install-editable-option' of github.com:mlouielu/pi…
mlouielu Jul 4, 2018
5d759a1
Consist to use double quote
mlouielu Jul 4, 2018
860a3ad
Merge branch 'master' of https://github.com/pypa/pipenv into make-ins…
mlouielu Jul 4, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix editable
  • Loading branch information
@mlouielu
mlouielu committed Jun 28, 2018
commit cbf398fbdeea18f05916572ede586a9d632afb1f
8 changes: 4 additions & 4 deletions pipenv/core.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1508,7 +1508,7 @@ def pip_install(


def pip_download(package_name):
cache_dir = Path(PIPENV_CACHE_DIR)
cache_dir = Path(PIPENV_CACHE_DIR)
pip_config = {
'PIP_CACHE_DIR': fs_str(cache_dir.as_posix()),
'PIP_WHEEL_DIR': fs_str(cache_dir.joinpath('wheels').as_posix()),
Expand Down Expand Up @@ -1891,9 +1891,9 @@ def do_install(
project.add_package_to_pipfile(req)
# Assign editable (-e) to following package_name.
if editable:
package_name = editable
# capture indexes and extra indexes
package_name = ' '.join(['-e', editable])
more_packages = list(more_packages)
# capture indexes and extra indexes
line = [package_name] + more_packages
index_indicators = ['-i', '--index', '--extra-index-url']
index, extra_indexes = None, None
Expand Down Expand Up @@ -1994,7 +1994,7 @@ def do_install(
click.echo('{0}: {1}'.format(crayons.red('WARNING'), e))
requirements_directory.cleanup()
sys.exit(1)
if converted.is_vcs and not converted.editable:
if converted.is_vcs and not editable:
click.echo(
'{0}: You installed a VCS dependency in non–editable mode. '
'This will work fine, but sub-dependencies will not be resolved by {1}.'
Expand Down