Skip to content

Default max-positional-args to max-args#8998

Merged
charliermarsh merged 1 commit intomainfrom
charlie/args
Dec 4, 2023
Merged

Default max-positional-args to max-args#8998
charliermarsh merged 1 commit intomainfrom
charlie/args

Conversation

@charliermarsh
Copy link
Copy Markdown
Member

@charliermarsh charliermarsh commented Dec 4, 2023

No description provided.

@charliermarsh charliermarsh added the configuration Related to settings and configuration label Dec 4, 2023
@charliermarsh charliermarsh changed the title Default max-positional-args to max-args Default max-positional-args to max-args Dec 4, 2023
@charliermarsh charliermarsh enabled auto-merge (squash) December 4, 2023 18:51
@charliermarsh charliermarsh merged commit 93258e8 into main Dec 4, 2023
@charliermarsh charliermarsh deleted the charlie/args branch December 4, 2023 19:02
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 4, 2023

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

ℹ️ ecosystem check detected linter changes. (+0 -97 violations, +0 -0 fixes in 41 projects)

milvus-io/pymilvus (+0 -38 violations, +0 -0 fixes) 

ruff check --no-cache --exit-zero --preview

- pymilvus/bulk_writer/bulk_import.py:79:5: PLR0917 Too many positional arguments: (7/5)
- pymilvus/bulk_writer/remote_bulk_writer.py:36:13: PLR0917 Too many positional arguments: (10/5)
- pymilvus/bulk_writer/remote_bulk_writer.py:58:9: PLR0917 Too many positional arguments: (6/5)
- pymilvus/client/abstract.py:389:9: PLR0917 Too many positional arguments: (6/5)
- pymilvus/client/grpc_handler.py:1281:9: PLR0917 Too many positional arguments: (6/5)
- pymilvus/client/grpc_handler.py:1295:9: PLR0917 Too many positional arguments: (6/5)
- pymilvus/client/grpc_handler.py:1338:9: PLR0917 Too many positional arguments: (6/5)
- pymilvus/client/grpc_handler.py:1595:9: PLR0917 Too many positional arguments: (7/5)
- pymilvus/client/grpc_handler.py:1617:9: PLR0917 Too many positional arguments: (7/5)
- pymilvus/client/grpc_handler.py:1648:9: PLR0917 Too many positional arguments: (6/5)
- pymilvus/client/grpc_handler.py:1706:9: PLR0917 Too many positional arguments: (6/5)
- pymilvus/client/grpc_handler.py:590:9: PLR0917 Too many positional arguments: (6/5)
- pymilvus/client/grpc_handler.py:712:9: PLR0917 Too many positional arguments: (11/5)
- pymilvus/client/prepare.py:1009:9: PLR0917 Too many positional arguments: (7/5)
- pymilvus/client/prepare.py:406:9: PLR0917 Too many positional arguments: (6/5)
- pymilvus/client/prepare.py:426:9: PLR0917 Too many positional arguments: (6/5)
- pymilvus/client/prepare.py:557:9: PLR0917 Too many positional arguments: (6/5)
- pymilvus/client/prepare.py:568:9: PLR0917 Too many positional arguments: (10/5)
- pymilvus/client/prepare.py:707:9: PLR0917 Too many positional arguments: (6/5)
... 19 additional changes omitted for project

pypa/pip (+0 -59 violations, +0 -0 fixes) 

ruff check --no-cache --exit-zero --preview

- src/pip/_internal/cli/req_command.py:271:9: PLR0917 Too many positional arguments: (9/5)
- src/pip/_internal/cli/req_command.py:326:9: PLR0917 Too many positional arguments: (12/5)
- src/pip/_internal/index/collector.py:260:9: PLR0917 Too many positional arguments: (6/5)
- src/pip/_internal/index/package_finder.py:120:9: PLR0917 Too many positional arguments: (7/5)
- src/pip/_internal/index/package_finder.py:393:9: PLR0917 Too many positional arguments: (7/5)
- src/pip/_internal/index/package_finder.py:428:9: PLR0917 Too many positional arguments: (7/5)
- src/pip/_internal/index/package_finder.py:597:9: PLR0917 Too many positional arguments: (7/5)
- src/pip/_internal/locations/__init__.py:230:5: PLR0917 Too many positional arguments: (6/5)
- src/pip/_internal/locations/_distutils.py:115:5: PLR0917 Too many positional arguments: (6/5)
- src/pip/_internal/locations/_distutils.py:35:5: PLR0917 Too many positional arguments: (6/5)
- src/pip/_internal/locations/_sysconfig.py:124:5: PLR0917 Too many positional arguments: (6/5)
- src/pip/_internal/metadata/base.py:645:9: PLR0917 Too many positional arguments: (6/5)
- src/pip/_internal/models/link.py:197:9: PLR0917 Too many positional arguments: (8/5)
- src/pip/_internal/models/scheme.py:19:9: PLR0917 Too many positional arguments: (6/5)
- src/pip/_internal/models/selection_prefs.py:24:9: PLR0917 Too many positional arguments: (6/5)
- src/pip/_internal/network/session.py:211:9: PLR0917 Too many positional arguments: (7/5)
- src/pip/_internal/operations/freeze.py:26:5: PLR0917 Too many positional arguments: (7/5)
- src/pip/_internal/operations/install/wheel.py:426:5: PLR0917 Too many positional arguments: (8/5)
- src/pip/_internal/operations/install/wheel.py:713:5: PLR0917 Too many positional arguments: (8/5)
- src/pip/_internal/operations/prepare.py:140:5: PLR0917 Too many positional arguments: (6/5)
- src/pip/_internal/operations/prepare.py:216:9: PLR0917 Too many positional arguments: (15/5)
- src/pip/_internal/req/__init__.py:37:5: PLR0917 Too many positional arguments: (8/5)
- src/pip/_internal/req/req_file.py:103:9: PLR0917 Too many positional arguments: (6/5)
- src/pip/_internal/req/req_file.py:208:5: PLR0917 Too many positional arguments: (6/5)
- src/pip/_internal/req/req_file.py:85:9: PLR0917 Too many positional arguments: (7/5)
- src/pip/_internal/req/req_install.py:72:9: PLR0917 Too many positional arguments: (8/5)
- src/pip/_internal/req/req_install.py:809:9: PLR0917 Too many positional arguments: (8/5)
- src/pip/_internal/resolution/legacy/resolver.py:120:9: PLR0917 Too many positional arguments: (12/5)
- src/pip/_internal/resolution/resolvelib/candidates.py:141:9: PLR0917 Too many positional arguments: (7/5)
- src/pip/_internal/resolution/resolvelib/candidates.py:250:9: PLR0917 Too many positional arguments: (6/5)
... 29 additional changes omitted for project

Changes by rule (1 rules affected)

code total + violation - violation + fix - fix
PLR0917 97 0 97 0 0

@miccal miccal mentioned this pull request Dec 5, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

configuration Related to settings and configuration

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@charliermarsh