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

record the packages needed to test #74

Closed
wants to merge 1 commit into from
Closed

Conversation

parmentelat
Copy link
Contributor

when setting up my test env again I recorded the dependencies
I'm not sure where this fits best, I have a feeling that pyproject.toml could be the right place for that, but I can't accurate information, so tests/requirements.txt may do the job

no big deal really if you think this is out of scope...

@jwg4
Copy link
Owner

jwg4 commented Nov 15, 2023

hi, thanks for this.

all the requirements that you mentioned are in the pyproject.toml file (except poetry, since using the file at all depends on poetry). the way to run tests based on this is to do:

poetry install # installs both the dependencies and the package itself from source code
followed by
poetry run test # runs all the tests, inside an environment created by poetry

you can also do poetry run quicktest to run the quick tests only, or poetry run python if you want to have a python shell which has the package and all the requirements set up for you.

let me know if this is what you need and I'll add the info to the documentation

@parmentelat
Copy link
Contributor Author

Hi
sorry I had no exposure to poetry before, and I did not take the time to do the homework
please forget about this PR then

@parmentelat parmentelat deleted the req branch November 19, 2023 13:42
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.

2 participants