Skip to content

chore(pyproject): disable COM812 ruff rule - #24

Merged
mirzak merged 1 commit into
mainfrom
ruff-warning
Oct 20, 2025
Merged

chore(pyproject): disable COM812 ruff rule#24
mirzak merged 1 commit into
mainfrom
ruff-warning

Conversation

@mirzak

@mirzak mirzak commented Oct 20, 2025

Copy link
Copy Markdown
Contributor

From the docs [1]:

We recommend against using this rule alongside the formatter.
The formatter enforces consistent use of trailing commas,
making the rule redundant.

Fixes the following warning:

warning: The following rule may cause conflicts when used with
the formatter: `COM812`. To avoid unexpected behavior, we
recommend disabling this rule, either by removing it from the
`lint.select` or `lint.extend-select` configuration, or adding
it to the `lint.ignore` configuration.
  1. https://docs.astral.sh/ruff/rules/missing-trailing-comma/

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR addresses a configuration conflict between Ruff's linting rules and formatter by disabling the COM812 rule (missing-trailing-comma) as recommended by the tool's documentation.

Key Changes:

  • Added ignore section to Ruff lint configuration to exclude COM812 rule

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread pyproject.toml Outdated
From the docs [1]:

> We recommend against using this rule alongside the formatter.
  The formatter enforces consistent use of trailing commas,
  making the rule redundant.

Fixes the following warning:

    warning: The following rule may cause conflicts when used with
    the formatter: `COM812`. To avoid unexpected behavior, we
    recommend disabling this rule, either by removing it from the
    `lint.select` or `lint.extend-select` configuration, or adding
    it to the `lint.ignore` configuration.

1. https://docs.astral.sh/ruff/rules/missing-trailing-comma/

Signed-off-by: Mirza Krak <mirza@mkrak.org>
@mirzak
mirzak merged commit 9f912d7 into main Oct 20, 2025
8 checks passed
@mirzak
mirzak deleted the ruff-warning branch October 20, 2025 16:52
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.

3 participants