Skip to content

build: upgrade Python linter ruff, add rules ASYNC,PERF - #59984

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
cclauss:upgrade-python-linter-ruff
Oct 8, 2025
Merged

build: upgrade Python linter ruff, add rules ASYNC,PERF#59984
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
cclauss:upgrade-python-linter-ruff

Conversation

@cclauss

@cclausscclauss commented Sep 23, 2025

Copy link
Copy Markdown
Contributor

@nodejs-github-botnodejs-github-bot added build Issues and PRs related to build files or the CI. i18n-api Issues and PRs related to the i18n implementation. icu Issues and PRs related to the ICU dependency. needs-ci PRs that need a full CI run. python PRs and issues that require attention from people who are familiar with Python. test Issues and PRs related to the tests. tools Issues and PRs related to the tools directory. labels Sep 23, 2025
@codecov

codecovBot commented Sep 23, 2025•
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.43%. Comparing base (4396cf2) to head (fc6e77a).
⚠️ Report is 119 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #59984 +/- ##
==========================================
- Coverage 88.46% 88.43% -0.03% 
==========================================
Files 703 703 Lines 207368 207368 Branches 39983 39983 ==========================================
- Hits 183438 183394 -44 - Misses 15933 15990 +57 + Partials 7997 7984 -13

see 45 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@cclauss
cclaussforce-pushed the upgrade-python-linter-ruff branch from cdf5364 to fc6e77aCompareSeptember 23, 2025 10:16
@lpincalpinca added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 23, 2025
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 23, 2025
@nodejs-github-bot

nodejs-github-bot commented Sep 23, 2025

Copy link
Copy Markdown
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/69341/

@MoLowMoLow added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 8, 2025
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 8, 2025
@nodejs-github-bot
nodejs-github-bot merged commit 0fb0406 into nodejs:mainOct 8, 2025
60 checks passed
@nodejs-github-bot

nodejs-github-bot commented Oct 8, 2025

Copy link
Copy Markdown
Collaborator

Landed in 0fb0406

@cclauss
cclauss deleted the upgrade-python-linter-ruff branch October 8, 2025 07:40
@RafaelGSSRafaelGSS mentioned this pull request Oct 13, 2025
Merged
9 tasks
targos pushed a commit that referenced this pull request Nov 18, 2025
@cclauss@targos
build: upgrade Python linter ruff, add rules ASYNC,PERF
acbc8ca
PR-URL: #59984
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
@github-actionsgithub-actionsBot mentioned this pull request Dec 9, 2025
Merged
This was referenced Jan 3, 2026
Closed
Closed
aduh95 pushed a commit that referenced this pull request Feb 17, 2026
@cclauss@aduh95
build: upgrade Python linter ruff, add rules ASYNC,PERF
003fcdf
PR-URL: #59984
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
aduh95 pushed a commit that referenced this pull request Feb 19, 2026
@cclauss@aduh95
build: upgrade Python linter ruff, add rules ASYNC,PERF
ad51085
PR-URL: #59984
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
@aduh95aduh95 mentioned this pull request Feb 22, 2026
Merged
aduh95 pushed a commit that referenced this pull request Feb 23, 2026
@cclauss@aduh95
build: upgrade Python linter ruff, add rules ASYNC,PERF
9408c44
PR-URL: #59984
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lpincalpincalpinca approved these changes
@anonriganonriganonrig approved these changes
@MoLowMoLowMoLow approved these changes
@legendecaslegendecaslegendecas approved these changes

Assignees

No one assigned

Labels

buildIssues and PRs related to build files or the CI.i18n-apiIssues and PRs related to the i18n implementation.icuIssues and PRs related to the ICU dependency.needs-ciPRs that need a full CI run.pythonPRs and issues that require attention from people who are familiar with Python.testIssues and PRs related to the tests.toolsIssues and PRs related to the tools directory.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@cclauss@nodejs-github-bot@lpinca@anonrig@MoLow@legendecas