Skip to content

Ruff: Ignore PLR0913 violations (too-many-arguments) in the src directory - #4748

Merged
seisman merged 1 commit into
mainfrom
ruff/PLR0913
Jul 23, 2026
Merged

Ruff: Ignore PLR0913 violations (too-many-arguments) in the src directory#4748
seisman merged 1 commit into
mainfrom
ruff/PLR0913

Conversation

@seisman

@seisman seisman commented Jul 22, 2026

Copy link
Copy Markdown
Member

Currently, the maximum allowed number of arguments for functions and methods is set to 10:

max-args=10

This limit is insufficient for PyGMT functions and methods that wrap GMT modules. At present, we add noqa: PLR0913 to suppress this lint violation in nearly every file within pygmt/src/.

We could set max-args to a larger value like 20, but this would permit longer argument lists for non-wrapper functions and methods, which is undesirable.

Instead, this PR suppresses the PLR0913 violations selectively for all files under the src directory.

@seisman seisman added this to the 0.20.0 milestone Jul 22, 2026
@seisman seisman added maintenance Boring but important stuff for the core devs skip-changelog Skip adding Pull Request to changelog needs review This PR has higher priority and needs review. labels Jul 22, 2026
@seisman seisman added final review call This PR requires final review and approval from a second reviewer and removed needs review This PR has higher priority and needs review. labels Jul 22, 2026
@seisman
seisman merged commit ef89b0c into main Jul 23, 2026
25 of 28 checks passed
@seisman
seisman deleted the ruff/PLR0913 branch July 23, 2026 02:58
@seisman seisman removed the final review call This PR requires final review and approval from a second reviewer label Jul 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Boring but important stuff for the core devs skip-changelog Skip adding Pull Request to changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants