search

arXiv / arxiv-submission-ui

User interface of NG submit system.
MIT License
2 stars 6 forks source link

ARXIVNG-2982: Use 'pipenv sync' instead of 'pipenv install'. #177

Closed DavidLFielding closed 4 years ago

DavidLFielding commented 4 years ago

'pipenv install' misses specific package dependencies installed in a developer's local virtual environment. 'pipenv sync' installs the same detailed list of packages contained in the Pipfile.lock file that were installed in the environment of the last developer to commit the Pipfile.lock file.

I've made a minor change to the Dockerfile to use 'pipenv sync'.

The developer team should discuss further whether 'pipenv sync' is the preferred way to go.

This change resolves the missing package error detailed in ticket ARXIVNG-2982.

DavidLFielding commented 4 years ago

I'm going to go ahead and merge this PR as it seems sync reproduced the previous developers virtual environment more accurately than install does.

mhl10 commented 4 years ago

This looked good, sorry to be late to the game.