-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
feat(ast-spec): bring Node
objects in line with ESTree
#3771
Conversation
Thanks for the PR, @MichaelDeBoey! 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. As a thank you, your profile/company logo will be added to our main README which receives thousands of unique visitors per day. |
166c9da
to
bd887d5
Compare
@bradzacher The PR as it is right now isn't a breaking change, as we still expose Was planning on doing a follow-up PR to the |
a5cbc47
to
ba26318
Compare
ba26318
to
be6d4da
Compare
Codecov Report
@@ Coverage Diff @@
## v5 #3771 +/- ##
==========================================
- Coverage 93.40% 92.70% -0.71%
==========================================
Files 151 331 +180
Lines 8084 11640 +3556
Branches 2563 3291 +728
==========================================
+ Hits 7551 10791 +3240
- Misses 181 371 +190
- Partials 352 478 +126
Flags with carried forward coverage won't be shown. Click here to find out more.
|
let's just target this at v5 and avoid the interim migration step. |
@bradzacher Do I remove the |
Yup! Remove it and rebase on the v5 branch |
8f540b7
to
4e87ef9
Compare
@bradzacher It should be good to go now 🙂 |
BREAKING CHANGE: `LineAndColumnData` is renamed to `Position`
4e87ef9
to
275eb0b
Compare
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.
thanks!
Node
objects in line with ESTreeNode
objects in line with ESTree
https://github.com/estree/estree/blob/master/es5.md#node-objects
https://eslint.org/docs/developer-guide/working-with-custom-parsers#the-ast-specification
BREAKING CHANGE:
LineAndColumnData
is renamed toPosition