Skip to content

Fix ruff PLR0917 lint failure in CI - #922

Draft
ianhelle with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-lint-job-failure
Draft

Fix ruff PLR0917 lint failure in CI#922
ianhelle with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-lint-job-failure

Conversation

Copilot AI commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

The "lint (3.12)" CI job fails because ruff 0.16.0 introduced PLR0917 (Too many positional arguments), which fires on three existing functions that already carry # pylint: disable=too-many-arguments — the high argument counts are intentional design.

Change

  • Added PLR0917 to the ruff ignore list in pyproject.toml, alongside the existing PLR0913 suppression:
"PLR0913", # Too many arguments
"PLR0917", # Too many positional arguments

Affected functions (none modified):

  • compute_likelihood_windows_in_session — 11 positional args
  • rarest_window_session — 11 positional args
  • plot_nx_graph — 12 positional args

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI changed the title [WIP] Fix the failing GitHub Actions job lint (3.12) Fix ruff PLR0917 lint failure in CI Jul 28, 2026
Copilot AI requested a review from ianhelle July 28, 2026 22:53
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