Skip to content

Add linting rules for ruff - #22741

Merged
Brett Cannon (brettcannon) merged 21 commits into
microsoft:mainfrom
brettcannon:ruff-rules
Mar 12, 2024
Merged

Add linting rules for ruff#22741
Brett Cannon (brettcannon) merged 21 commits into
microsoft:mainfrom
brettcannon:ruff-rules

Conversation

@brettcannon

Copy link
Copy Markdown
Member

No description provided.

@brettcannon Brett Cannon (brettcannon) added the debt Code quality issues label Jan 11, 2024
Comment thread .github/actions/lint/action.yml
Comment thread pythonFiles/pyproject.toml Outdated
Co-authored-by: Karthik Nadig <kanadig@microsoft.com>
@aiday-mar Aiday Marlen Kyzy (aiday-mar) modified the milestones: December / January 2024, February 2024 Jan 24, 2024
@brettcannon

Copy link
Copy Markdown
Member Author

Another option here is to go w/ the default ruleset and just specify the docstring rules.

@brettcannon

Copy link
Copy Markdown
Member Author

Eleanor Boyd (@eleanorjboyd) any idea why these tests are failing when I didn't make any semantic changes or touch the .data directory?

@brettcannon
Brett Cannon (brettcannon) merged commit 7ea5584 into microsoft:main Mar 12, 2024
@brettcannon
Brett Cannon (brettcannon) deleted the ruff-rules branch March 12, 2024 19:25
Anthony Kim (anthonykim1) pushed a commit to anthonykim1/vscode-python that referenced this pull request Mar 14, 2024
Co-authored-by: Karthik Nadig <kanadig@microsoft.com>
Co-authored-by: eleanorjboyd <eleanorboyd@microsoft.com>
Wes McKinney (wesm) pushed a commit to posit-dev/positron that referenced this pull request Apr 5, 2024
Co-authored-by: Karthik Nadig <kanadig@microsoft.com>
Co-authored-by: eleanorjboyd <eleanorboyd@microsoft.com>
Wes McKinney (wesm) pushed a commit to posit-dev/positron that referenced this pull request Apr 8, 2024
Co-authored-by: Karthik Nadig <kanadig@microsoft.com>
Co-authored-by: eleanorjboyd <eleanorboyd@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

debt Code quality issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants