Skip to content

Apply Scientific Python Library Development Guide Repo-Review suggestions#1528

Merged
HaoZeke merged 5 commits intoairspeed-velocity:mainfrom
DimitriPapadopoulos:repo-review
Sep 13, 2025
Merged

Apply Scientific Python Library Development Guide Repo-Review suggestions#1528
HaoZeke merged 5 commits intoairspeed-velocity:mainfrom
DimitriPapadopoulos:repo-review

Conversation

@DimitriPapadopoulos
Copy link
Contributor

GH200: Maintained by Dependabot
All projects should have a .github/dependabot.yml file to support at least GitHub Actions regular updates.

https://docs.github.com/en/code-security/dependabot/working-with-dependabot/keeping-your-actions-up-to-date-with-dependabot
PP302: Sets a minimum pytest to at least 6
Must have a `minversion=`, and must be at least 6 (first version to support `pyproject.toml` configuration).
PP303: Sets the test paths
The `testpaths` setting should be set to a reasonable default.
PP304: Sets the log level in pytest
`log_cli_level` should be set. This will allow logs to be displayed on failures.
PP306: Specifies strict config
`--strict-config` should be in `addopts = [...]`. This forces an error if a config setting is misspelled.

PP307: Specifies strict markers
`--strict-markers` should be in `addopts = [...]`. This forces all markers to be specified in config, avoiding misspellings.

PP308: Specifies useful pytest summary
An explicit summary flag like `-ra` should be in `addopts = [...]` (print summary of all fails/errors).
@DimitriPapadopoulos DimitriPapadopoulos changed the title Start applying Scientific Python Library Development Guide Repo-Review suggestions Apply Scientific Python Library Development Guide Repo-Review suggestions Sep 13, 2025
@HaoZeke HaoZeke merged commit 5ea038a into airspeed-velocity:main Sep 13, 2025
14 checks passed
@DimitriPapadopoulos DimitriPapadopoulos deleted the repo-review branch September 13, 2025 14:09
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