fix(scope-manager): export ClassStaticBlockScope#12460
Conversation
|
Thanks for the PR, @zaewc! 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. |
✅ Deploy Preview for typescript-eslint ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
View your CI Pipeline Execution ↗ for commit 56ac2ea
💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗ ☁️ Nx Cloud last updated this comment at |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #12460 +/- ##
==========================================
+ Coverage 87.06% 91.24% +4.17%
==========================================
Files 514 472 -42
Lines 16616 14503 -2113
Branches 5193 4595 -598
==========================================
- Hits 14467 13233 -1234
+ Misses 1461 752 -709
+ Partials 688 518 -170
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
Hi @zaewc, thanks for the update! Just a quick heads-up: please try to avoid force-pushing moving forward. Rewriting history makes it harder for us to track incremental changes between reviews. Since we squash merge anyway, there is no need to keep the commit history "clean" on this branch. Standard pushes are much preferred! |
Thanks for the heads-up and sorry about the force push I rewrote the branch to remove the temporary test commit after addressing the review, but I understand that this makes review history harder to follow. I’ll avoid force-pushing on future updates and use regular follow-up commits instead. |
@zaewc note https://typescript-eslint.io/contributing/ai-policy 🙂 |
ee39789
into
typescript-eslint:main

PR Checklist
Overview
ClassStaticBlockScopewas part of the internal scope model but missing from the public@typescript-eslint/scope-managerbarrel exportThis PR exports it and adds a regression test to ensure it can be imported from the package entrypoint