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.

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

Merged
merged 8 commits into from Apr 5, 2022
Merged

Conversation

jnussbaum
Copy link
Collaborator

resolves DEV-764

@jnussbaum jnussbaum changed the title Wip/dev 764 change to pipenv chore: change to pipenv (DEV-764) Mar 31, 2022
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Makefile Show resolved Hide resolved
@jnussbaum
Copy link
Collaborator Author

jnussbaum commented Apr 1, 2022

@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?

Copy link
Collaborator

@BalduinLandolt BalduinLandolt left a 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

Makefile Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Copy link

@irinaschubert irinaschubert left a 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:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
This works because there are two files defining the dependencies:
There are two files defining the dependencies:

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
@sonarcloud
Copy link

sonarcloud bot commented Apr 5, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@jnussbaum jnussbaum merged commit 6c44688 into main Apr 5, 2022
@jnussbaum jnussbaum deleted the wip/dev-764-change-to-pipenv branch April 5, 2022 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants