typescript-eslint/typescript-eslintPublic

NotificationsYou must be signed in to change notification settings
Fork2.8k
Star15.8k

chore: speed up semantic-breaking-change-pr.yml#11286

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

JoshuaKGoldberg merged 13 commits intotypescript-eslint:mainfromdeveloper-bandi:feat/semantic-breaking-change-pr-speed-up

Jun 16, 2025

Merged

chore: speed up semantic-breaking-change-pr.yml#11286

JoshuaKGoldberg merged 13 commits intotypescript-eslint:mainfromdeveloper-bandi:feat/semantic-breaking-change-pr-speed-up

Jun 16, 2025

+57 −283

Conversation

Copy link

Contributor

developer-bandi commentedJun 8, 2025•
edited
Loading

PR Checklist

Addresses an existing open issue:fixesRepo: Speed up semantic-breaking-change-pr.yml #11206
That issue was marked asaccepting prs
Steps inContributing were taken

Overview

I reconfigured it using actions/github-script since it doesn't require installing the required packages.

and script is excute to main branch, so if we check test result, seehttps://github.com/typescript-eslint/typescript-eslint/actions/runs/15559798943/job/43808999326

The execution time was reduced from approximately 32 seconds -> 7 seconds.

developer-bandi added2 commits

June 8, 2025 21:29

feat(breaking-pr-check): Refactored to breaking-pr-check and deleted …

d3f8679

…index.js file

feat: change pr code

1897bf9

Copy link

Contributor

typescript-eslintbot commentedJun 8, 2025

Thanks for the PR,@developer-bandi!

typescript-eslint is a 100% community driven project, and we are incredibly grateful that you are contributing to that community.

The core maintainers work on this in their personal time, so please understand that it may not be possible for them to review your work immediately.

Thanks again!

🙏Please, if you or your company is finding typescript-eslint valuable, help us sustain the project by sponsoring it transparently on https://opencollective.com/typescript-eslint.

Copy link

netlifybot commentedJun 8, 2025•
edited
Loading

✅ Deploy Preview fortypescript-eslint ready!

Name	Link
🔨 Latest commit	`d32357f`
🔍 Latest deploy log	https://app.netlify.com/projects/typescript-eslint/deploys/68501e57a2e5fe0008d5a11e
😎 Deploy Preview	https://deploy-preview-11286--typescript-eslint.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	1 paths audited Performance: 72 (🔴 down 27 from production) Accessibility: 100 (no change from production) Best Practices: 100 (no change from production) SEO: 100 (🟢 up 2 from production) PWA: 80 (no change from production) View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to yourNetlify project configuration.

developer-bandi marked this pull request as draft

June 8, 2025 12:33

Copy link

nx-cloudbot commentedJun 8, 2025•
edited
Loading

View yourCI Pipeline Execution ↗ for commitd32357f.

Command	Status	Duration	Result
`nx test eslint-plugin --coverage=false`	✅ Succeeded	5m 8s	View ↗
`nx run-many -t typecheck`	✅ Succeeded	2m 15s	View ↗
`nx run integration-tests:test`	✅ Succeeded	<1s	View ↗
`nx run types:build`	✅ Succeeded	<1s	View ↗
`nx run-many -t lint`	✅ Succeeded	10s	View ↗
`nx test eslint-plugin-internal --coverage=false`	✅ Succeeded	<1s	View ↗
`nx test typescript-estree --coverage=false`	✅ Succeeded	<1s	View ↗
`nx run generate-configs`	✅ Succeeded	10s	View ↗
`Additional runs (27)`	✅ Succeeded	...	View ↗

☁️Nx Cloud last updated this comment at2025-06-16 13:49:09 UTC

refactor: remove unused dependencies

a5e66db

developer-bandi changed the title~~chore: Speed up semantic-breaking-change-pr.yml~~chore: speed up semantic-breaking-change-pr.yml

Jun 9, 2025

developer-bandi added8 commits

June 10, 2025 06:06

test: rule test

d3ca756

chore: add test code

ca3afbf

chore: add test logic

712411d

chore: rename action and remove obsolete workflow file

34fc314

feat: add permission

0aa12b7

fix: token

e94356f

fix: revert test code & lock file update

1e77541

fix: remove test code

375efef

developer-bandi marked this pull request as ready for review

June 10, 2025 13:05

Copy link

codecovbot commentedJun 10, 2025•
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.84%. Comparing base(f9d4d10) to head(d32357f).
Report is 5 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@##             main   #11286   +/-   ##=======================================  Coverage   90.84%   90.84%           =======================================  Files         501      501             Lines       50911    50911             Branches     8383     8383           =======================================  Hits        46248    46248             Misses       4648     4648             Partials       15       15

Flag	Coverage Δ
unittest	`90.84% <ø> (ø)`

Flags with carried forward coverage won't be shown.Click here to find out more.

🚀 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.