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

Assorted repo-review suggestions #1497

Merged
merged 8 commits into from
Aug 6, 2024
Merged

Commits on Aug 5, 2024

  1. 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).
    DimitriPapadopoulos committed Aug 5, 2024
    Configuration menu
    Copy the full SHA
    c8d4f68 View commit details
    Browse the repository at this point in the history
  2. PP303: Sets the test paths

    Apply repo-review rule PP303:
    
    	The `testpaths` setting should be set to a reasonable default.
    DimitriPapadopoulos committed Aug 5, 2024
    Configuration menu
    Copy the full SHA
    b4df6b9 View commit details
    Browse the repository at this point in the history
  3. 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.
    DimitriPapadopoulos committed Aug 5, 2024
    Configuration menu
    Copy the full SHA
    47c1925 View commit details
    Browse the repository at this point in the history
  4. 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.
    DimitriPapadopoulos committed Aug 5, 2024
    Configuration menu
    Copy the full SHA
    aa26f14 View commit details
    Browse the repository at this point in the history
  5. 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.
    DimitriPapadopoulos committed Aug 5, 2024
    Configuration menu
    Copy the full SHA
    caac745 View commit details
    Browse the repository at this point in the history
  6. 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).
    DimitriPapadopoulos committed Aug 5, 2024
    Configuration menu
    Copy the full SHA
    d04b578 View commit details
    Browse the repository at this point in the history
  7. PC191: Ruff show fixes if fixes enabled

    Apply repo-review rule PC191:
    
    	If --fix is present, --show-fixes must be too.
    DimitriPapadopoulos committed Aug 5, 2024
    Configuration menu
    Copy the full SHA
    44b5d12 View commit details
    Browse the repository at this point in the history

Commits on Aug 6, 2024

  1. Configuration menu
    Copy the full SHA
    2eb09ea View commit details
    Browse the repository at this point in the history