Skip to content

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

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

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

Conversation

@charliermarsh

@charliermarsh charliermarsh commented Dec 4, 2023

Copy link
Copy Markdown
Member

No description provided.

@charliermarsh charliermarsh added the configuration Related to settings and configuration of Ruff or ty 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

github-actions Bot commented Dec 4, 2023

Copy link
Copy Markdown
Contributor

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 of Ruff or ty

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@charliermarsh