🌐 AI搜索 & 代理 主页
Skip to content

Conversation

@KuSh
Copy link
Contributor

@KuSh KuSh commented Sep 6, 2025

PR Checklist

Overview

Enable eslint no-useless-rename rule on project, see discussion

Signed-off-by: Nicolas Le Cam <niko.lecam@gmail.com>
@typescript-eslint
Copy link
Contributor

Thanks for the PR, @KuSh!

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 Sep 6, 2025

Deploy Preview for typescript-eslint ready!

Name Link
🔨 Latest commit 91eb032
🔍 Latest deploy log https://app.netlify.com/projects/typescript-eslint/deploys/68bc028b5863da00084c2b6a
😎 Deploy Preview https://deploy-preview-11576--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: 91 (🔴 down 6 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.

@KuSh KuSh changed the title chore: enable no-useless-rename rule on project chore: enable ESLint no-useless-rename rule on project Sep 6, 2025
@nx-cloud
Copy link

nx-cloud bot commented Sep 6, 2025

View your CI Pipeline Execution ↗ for commit 91eb032

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

☁️ Nx Cloud last updated this comment at 2025-09-06 09:55:32 UTC

Copy link
Member

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Choose a reason for hiding this comment

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

🧹 thanks!!

@JoshuaKGoldberg JoshuaKGoldberg merged commit b44fde8 into typescript-eslint:main Sep 8, 2025
64 of 66 checks passed
@KuSh KuSh deleted the no-useless-rename branch September 8, 2025 15:35
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.

2 participants