Skip to content

Conversation

@typescript-eslint
Copy link
Contributor

Thanks for the PR, @JoshuaKGoldberg!

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.

@netlify
Copy link

netlify bot commented Dec 1, 2025

Deploy Preview for typescript-eslint ready!

Name Link
🔨 Latest commit 9a96603
🔍 Latest deploy log https://app.netlify.com/projects/typescript-eslint/deploys/692da2859a4d2b0008c386ce
😎 Deploy Preview https://deploy-preview-11812--typescript-eslint.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 87 (🔴 down 11 from production)
Accessibility: 97 (no change from production)
Best Practices: 100 (no change from production)
SEO: 92 (no change 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 your Netlify project configuration.

@nx-cloud
Copy link

nx-cloud bot commented Dec 1, 2025

View your CI Pipeline Execution ↗ for commit 9a96603

Command Status Duration Result
nx test eslint-plugin --coverage=false ✅ Succeeded 5m 11s View ↗
nx run-many -t lint ✅ Succeeded 3m 10s View ↗
nx run-many -t typecheck ✅ Succeeded 2m 26s View ↗
nx run types:build ✅ Succeeded 1s View ↗
nx run generate-configs ✅ Succeeded 7s View ↗
nx test typescript-estree --coverage=false ✅ Succeeded 21s View ↗
nx test eslint-plugin-internal --coverage=false ✅ Succeeded 9s View ↗
nx run integration-tests:test ✅ Succeeded 4s View ↗
Additional runs (29) ✅ Succeeded ... View ↗

☁️ Nx Cloud last updated this comment at 2025-12-01 14:29:17 UTC

@JoshuaKGoldberg JoshuaKGoldberg changed the title Revert "feat(eslint-plugin): [no-redundant-type-constituents] use assignability checking for redundancy checks" fix(eslint-plugin): revert "feat(eslint-plugin): [no-redundant-type-constituents] use assignability checking for redundancy checks" Dec 1, 2025
@JoshuaKGoldberg JoshuaKGoldberg changed the title fix(eslint-plugin): revert "feat(eslint-plugin): [no-redundant-type-constituents] use assignability checking for redundancy checks" fix(eslint-plugin): revert "[no-redundant-type-constituents] use assignability checking for redundancy checks" Dec 1, 2025
@codecov
Copy link

codecov bot commented Dec 1, 2025

Codecov Report

❌ Patch coverage is 95.72954% with 12 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.53%. Comparing base (f8b2bb4) to head (9a96603).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...plugin/src/rules/no-redundant-type-constituents.ts 96.40% 10 Missing ⚠️
...escript-estree/src/semantic-or-syntactic-errors.ts 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #11812      +/-   ##
==========================================
+ Coverage   90.49%   90.53%   +0.04%     
==========================================
  Files         523      523              
  Lines       53386    53097     -289     
  Branches     8919     8838      -81     
==========================================
- Hits        48311    48074     -237     
+ Misses       5061     5010      -51     
+ Partials       14       13       -1     
Flag Coverage Δ
unittest 90.53% <95.72%> (+0.04%) ⬆️

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

Files with missing lines Coverage Δ
...ages/eslint-plugin/src/rules/no-unsafe-argument.ts 93.24% <100.00%> (ø)
...nt-plugin/src/rules/related-getter-setter-pairs.ts 100.00% <ø> (ø)
...escript-estree/src/semantic-or-syntactic-errors.ts 2.32% <0.00%> (ø)
...plugin/src/rules/no-redundant-type-constituents.ts 97.64% <96.40%> (+4.22%) ⬆️

... and 1 file 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.

@JoshuaKGoldberg JoshuaKGoldberg merged commit 5ac66a6 into main Dec 1, 2025
74 of 76 checks passed
@JoshuaKGoldberg JoshuaKGoldberg deleted the revert-10744-7742-no-redundant-type-constituents branch December 1, 2025 14:49
jimmy-zhening-luo added a commit to jimmy-zhening-luo/linted-defaults that referenced this pull request Dec 2, 2025
…; re-enable the bugged lint rule:

eslint-plugin: revert: [no-redundant-type-constituents] use assignability checking for redundancy checks (typescript-eslint/typescript-eslint#10744) (typescript-eslint/typescript-eslint#11812)
jimmy-zhening-luo added a commit to jimmy-zhening-luo/linted that referenced this pull request Dec 2, 2025
…; re-enable the bugged lint rule:

- eslint-plugin: revert: [no-redundant-type-constituents] use assignability checking for redundancy checks (typescript-eslint/typescript-eslint#10744) (typescript-eslint/typescript-eslint#11812)
- chore: update pkg
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment