-
-
Notifications
You must be signed in to change notification settings - Fork 4
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
Use poetry for dependency management #69
Conversation
.github/workflows/test.yml
Outdated
@@ -7,8 +7,16 @@ jobs: | |||
- uses: actions/checkout@v3 | |||
- uses: actions/setup-python@v4 | |||
with: | |||
python-version: "3.8" | |||
python-version: "3.9" | |||
- name: Cache 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.
i think you can do
with:
- cache: 'poetry'
in the setup-python
action
.github/workflows/test.yml
Outdated
- name: Run tests | ||
run: pytest | ||
run: | | ||
pip install poetry |
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.
i dont know if this can cause issue but i remember there was another default way to install
@@ -130,7 +146,7 @@ Read our CONTRIBUTING.md then | |||
Create a virtual environment and install the package | |||
|
|||
``` | |||
make virtualenv | |||
poetry install |
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.
we need to update the documentation we have created as well
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.
Minor comments, make sure you update the documentation we have in mkdocs
Description
Follows #68 and resolves #67. Relevant to #66. Switches to poetry for dependency management.
Checklist