This repository has been archived by the owner on Feb 19, 2021. It is now read-only.
Finish migration to pipenv #661
Assignees
Labels
Comments
pitkley
added a commit
to pitkley/paperless
that referenced
this issue
May 19, 2020
In b1e6160 Pipenv was added to Paperless, adding a note to the changelog in eec8f09 that a switch to just using Pipenv was planned. This commit performs this switch. Having two sources for which dependencies should be installed (requirements.txt, Pipfile.lock) is very suboptimal, and the files have diverged since they have been added. This closes the-paperless-project#661.
|
My own and our official ansible role still use that file: https://github.com/the-paperless-project/ansible-role-paperless/blob/master/tasks/main.yml#L80 EDIT: Might be possible to fix this with pypa/pipenv#363 (comment) or by generating a |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Currently this repository contains both
requirements.txtandPipfile.lock. Those two files are not kept in sync, which means that depending on how a user installs Paperless, they will have a different set of dependencies available than if they had installed using a different method.To avoid confusion and ensure that our users always use the same set of dependencies, we should move to a single solution. Given the use of Pipenv within CI, removing
requirements.txtseems like the sensible option.(I mainly opened this issue to keep track of this topic myself. I'll work on migrating to Pipenv and will open a PR for this. If there are any opinions or concerns, please feel free to voice them -- either here or on the PR once it exists.)
The text was updated successfully, but these errors were encountered: