xarray-contrib / xarray-contrib/flox

Address repo-review comments

Open
#264 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
137
Forks
24
PR merge metrics
No merged PRs in 30d

Description

General
  • Detected build backend: setuptools.build_meta
  • Detected license(s): Apache Software License
?NameDescription
PY007 Supports an easy task runner (nox or tox)

Projects must have a noxfile.py or tox.ini to encourage new contributors.

PyProject
?NameDescription
PP304 Sets the log level in pytest

log_cli_level should be set. This will allow logs to be displayed on failures.

[tool.pytest.ini_options]
log_cli_level = "INFO"
PP305 Specifies xfail_strict

xfail_strict should be set. You can manually specify if a check should be strict when setting each xfail.

[tool.pytest.ini_options]
xfail_strict = true
PP306 Specifies strict config

--strict-config should be in addopts = [...]. This forces an error if a config setting is misspelled.

[tool.pytest.ini_options]
addops = ["-ra", "--strict-config", "--strict-markers"]
PP307 Specifies strict markers

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

[tool.pytest.ini_options]
addops = ["-ra", "--strict-config", "--strict-markers"]
PP308 Specifies useful pytest summary

-ra should be in addopts = [...] (print summary of all fails/errors).

[tool.pytest.ini_options]
addops = ["-ra", "--strict-config", "--strict-markers"]
PP309 Filter warnings specified

filterwarnings must be set (probably to at least ["error"]). Python will hide important warnings otherwise, like deprecations.

[tool.pytest.ini_options]
filterwarnings = ["error"]
Pre-commit
?NameDescription
PC111 Uses blacken-docs

Must have https://github.com/adamchainz/blacken-docs repo in .pre-commit-config.yaml

PC140 Uses mypy

Must have https://github.com/pre-commit/mirrors-mypy repo in .pre-commit-config.yaml

PC170 Uses PyGrep hooks (only needed if RST present)

Must have https://github.com/pre-commit/pygrep-hooks repo in .pre-commit-config.yaml

PC901 Custom pre-commit CI message

Should have something like this in .pre-commit-config.yaml:

ci:
  autoupdate_commit_msg: 'chore: update pre-commit hooks'
MyPy
?NameDescription
MY101 MyPy strict mode

Must have strict in the mypy config. MyPy is best with strict or nearly strict configuration. If you are happy with the strictness of your settings already, ignore this check or set strict = false explicitly.

[tool.mypy]
strict = true
Ruff
?NameDescription
RF101 Bugbear must be selected

Must select the flake8-bugbear B checks. Recommended:

[tool.ruff]
select = [
  "B",  # flake8-bugbear
]

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Review pyproject.toml and .pre-commit-config.yaml first, then compare the listed repo-review checks with the current configuration. Add the requested pytest, MyPy, Ruff, and pre-commit settings, and provide a noxfile.py or tox.ini for the task runner. Done means the affected repo-review checks pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
build-system, testing, tooling
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.