Skip to content

Avoid E275 if keyword is followed by a semicolon#12095

Merged
dhruvmanila merged 1 commit intomainfrom
dhruv/autofix-bug
Jun 28, 2024
Merged

Avoid E275 if keyword is followed by a semicolon#12095
dhruvmanila merged 1 commit intomainfrom
dhruv/autofix-bug

Conversation

@dhruvmanila
Copy link
Member

@dhruvmanila dhruvmanila commented Jun 28, 2024

fixes: #12094

@dhruvmanila dhruvmanila added the bug Something isn't working label Jun 28, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Jun 28, 2024

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

ℹ️ ecosystem check detected linter changes. (+0 -2 violations, +0 -0 fixes in 1 projects; 1 project error; 48 projects unchanged)

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

ruff check --no-cache --exit-zero --ignore RUF9 --output-format concise --preview

- examples/example_bulkinsert_json.py:209:13: E275 [*] Missing whitespace after keyword
- examples/example_bulkinsert_numpy.py:211:13: E275 [*] Missing whitespace after keyword

demisto/content (error) 

ruff check --no-cache --exit-zero --ignore RUF9 --output-format concise --preview

warning: The top-level linter settings are deprecated in favour of their counterparts in the `lint` section. Please update the following options in `pyproject.toml`:
  - 'ignore' -> 'lint.ignore'
  - 'select' -> 'lint.select'
  - 'unfixable' -> 'lint.unfixable'
  - 'per-file-ignores' -> 'lint.per-file-ignores'
warning: `PGH001` has been remapped to `S307`.
warning: `PGH002` has been remapped to `G010`.
warning: `PLR1701` has been remapped to `SIM101`.
ruff failed
  Cause: Selection of deprecated rule `E999` is not allowed when preview is enabled.

Changes by rule (1 rules affected)

code total + violation - violation + fix - fix
E275 2 0 2 0 0

@dhruvmanila dhruvmanila merged commit 47b2273 into main Jun 28, 2024
@dhruvmanila dhruvmanila deleted the dhruv/autofix-bug branch June 28, 2024 15:21
@BrewTestBot BrewTestBot mentioned this pull request Jul 5, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@charliermarsh charliermarsh charliermarsh approved these changes

Assignees

No one assigned

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Infinite loop in auto-fix between E203/E275

2 participants

@dhruvmanila @charliermarsh