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.
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
chore: change to pipenv (DEV-764) #177
Conversation
@BalduinLandolt @irinaschubert : In makefile, line 22, two arguments are passed to setup.py: sdist and bdist_wheel, but these are not used. Should we remove them? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See my comments for details, but overall it looks good
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just some minor details
|
||
To install from source run: | ||
This works because there are two files defining the dependencies: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This works because there are two files defining the dependencies: | |
There are two files defining the dependencies: |
Co-authored-by: irinaschubert <irina.schubert@dasch.swiss>
Kudos, SonarCloud Quality Gate passed!
|
resolves DEV-764