-
Notifications
You must be signed in to change notification settings - Fork 410
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
Assorted repo-review suggestions #1497
Commits on Aug 5, 2024
-
PP302: Sets a minimum pytest to at least 6
Apply repo-review rule PP302: Must have a minversion=, and must be at least 6 (first version to support pyproject.toml configuration).
Configuration menu - View commit details
-
Copy full SHA for c8d4f68 - Browse repository at this point
Copy the full SHA c8d4f68View commit details -
Apply repo-review rule PP303: The `testpaths` setting should be set to a reasonable default.
Configuration menu - View commit details
-
Copy full SHA for b4df6b9 - Browse repository at this point
Copy the full SHA b4df6b9View commit details -
PP304: Sets the log level in pytest
Apply repo-review rule PP304: log_cli_level should be set. This will allow logs to be displayed on failures.
Configuration menu - View commit details
-
Copy full SHA for 47c1925 - Browse repository at this point
Copy the full SHA 47c1925View commit details -
PP306: Specifies strict config
Apply repo-review rule PP306: --strict-config should be in addopts = [...]. This forces an error if a config setting is misspelled.
Configuration menu - View commit details
-
Copy full SHA for aa26f14 - Browse repository at this point
Copy the full SHA aa26f14View commit details -
PP307: Specifies strict markers
Apply repo-review rule PP307: --strict-markers should be in addopts = [...]. This forces all markers to be specified in config, avoiding misspellings.
Configuration menu - View commit details
-
Copy full SHA for caac745 - Browse repository at this point
Copy the full SHA caac745View commit details -
PP308: Specifies useful pytest summary
Apply repo-review rule PP308: An explicit summary flag like -ra should be in addopts = [...] (print summary of all fails/errors).
Configuration menu - View commit details
-
Copy full SHA for d04b578 - Browse repository at this point
Copy the full SHA d04b578View commit details -
PC191: Ruff show fixes if fixes enabled
Apply repo-review rule PC191: If --fix is present, --show-fixes must be too.
Configuration menu - View commit details
-
Copy full SHA for 44b5d12 - Browse repository at this point
Copy the full SHA 44b5d12View commit details
Commits on Aug 6, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 2eb09ea - Browse repository at this point
Copy the full SHA 2eb09eaView commit details