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

doc dir #1

Closed
wants to merge 7 commits into from
Closed

doc dir #1

wants to merge 7 commits into from

Conversation

penguinpee
Copy link
Owner

penguinpee and others added 7 commits May 21, 2024 14:24
Add `network` pytest mark to test functions that require network
connection to hit third party servers. Improve
`test_check_availability_invalid_downloader` to remove the need to use
network connection just to test an error raised after an invalid
argument to a custom downloader.

---------

Co-authored-by: Santiago Soler <[email protected]>
Change configuration of tests in GitHub Actions: use the latest x86
macos runner with the `oldest` dependencies and Python 3.7. Use the
latest macos runner (arm64) only against the `latest` and `optional`
requirements. This fixes CI failing because there are not Numpy binaries
in PyPI for arm64 and compatible with Python 3.7.
Include Pooch's optional dependencies to `environment.yml`. This allows
to run the full suite of tests after creating the environment and
installing Pooch with `make install`.
Using a variable in `pooch.downloaders` allows users to potentially
increase it if they require and allows us to update it easily if
required as well. Set it to 30s instead of 5s to allow for slow
connections.
Small bug fix release to fix the hard-coded timeout of requests
sessions.
The test code `pooch/tests` is installed but he data in `pooch/tests/data` 
are not. This makes it impossible to run tests on the installed package.
Add the appropriate setuptools configuration to make it happen.
The automatic discovery adds doc/ to the wheel triggered by the
presence of `conf.py` in that directory.
@penguinpee penguinpee closed this Jun 8, 2024
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.

4 participants