-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
chore: migrated shared-fixtures to ast-spec
#6436
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: migrated shared-fixtures to ast-spec
#6436
Conversation
|
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. |
✅ Deploy Preview for typescript-eslint ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #6436 +/- ##
==========================================
- Coverage 91.53% 90.66% -0.87%
==========================================
Files 371 371
Lines 12662 12662
Branches 3721 3721
==========================================
- Hits 11590 11480 -110
- Misses 754 845 +91
- Partials 318 337 +19
Flags with carried forward coverage won't be shown. Click here to find out more.
|
|
Let's do this:
Eg
After that I think we're in a good spot to land this? |
shared-fixtures to ast-spec
|
We'll also want to make sure these fixtures are automatically re-tested using the test tooling, so you'll need to update the test to include them |
|
Hmm, would you be upset if I moved |
|
ah yeah I guess the test isn't built to be relative to anything other than the |
bradzacher
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yas
PR Checklist
shared-fixturestests toast-specfixtures #6065Overview
Migrates all the old
shared-fixturesfixtures into the newast-specfixtures. Within that they're given the same folder name as their old fixture path.It would be better long-term to split these fixtures up as described in #6065:
...but there are ~375 of them, and I'd rather get them into
ast-specsooner. This way they'll be easier to split up later! If this PR is approved I can file a followup issue & find-and-replace the// TODOcomments in fixtures to refer to it.Most of the work in this PR was done in two steps:
327aa0c29: Ranyarn tsx tools/migrate-fixtures.tsto createunsorted-fixturesfrom the oldshared-fixturesbec1fe800: Deletedshared-fixturesaltogethertools/migrate-fixtures.ts