Skip to content

docs: add Google-style docs to diracx-cli and remove D100-D300 from per-file-ignores for it - #1033

Open
martynia wants to merge 3 commits into
DIRACGrid:mainfrom
martynia:janusz_rest2_googledocs_diracx-cli_1004
Open

docs: add Google-style docs to diracx-cli and remove D100-D300 from per-file-ignores for it#1033
martynia wants to merge 3 commits into
DIRACGrid:mainfrom
martynia:janusz_rest2_googledocs_diracx-cli_1004

Conversation

@martynia

@martynia martynia commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

This is a docstring fix PR for diracx-cli directory using a branched-off code from #1008.

The following changes and a verification procedure was performed:

  1. modify pyproject.toml by removing "ignores" from the relevant line: "diracx-cli/src/diracx/cli/**/*.py" = []
  2. verify failures: pixi run -e pre-commit pre-commit run ruff-check --files $(find diracx-cli -name '*.py')
  3. add google-style docstring to files in the directory in question one by one
  4. verify that ruff passes for a file fixed: i.e. pixi run -e pre-commit pre-commit run ruff-check --files $(find diracx-cli -name 'utils.py')
  5. when finished with all files verify that all files pass:pixi run -e pre-commit pre-commit run ruff-check --files $(find diracx-cli -name '*.py')

@martynia martynia changed the title docs: Add Google-style docs to diracx-cli and remove D100-D300 from per-file-ignores for it docs: add Google-style docs to diracx-cli and remove D100-D300 from per-file-ignores for it Sep 7, 2026
@martynia
martynia requested a review from aldbr September 7, 2026 12:05
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.

1 participant