Skip to content
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

[Breaking]: Don't return node.start & node.end #100

Merged
merged 1 commit into from
Oct 7, 2020

Conversation

MichaelDeBoey
Copy link
Contributor

@MichaelDeBoey MichaelDeBoey commented Jun 10, 2020

BREAKING CHANGE: Nodes don't return the non-standard node.start & node.end anymore

As discussed in #97 (comment), this is the PR with the breaking change of not providing node.start & node.end anymore.

BREAKING CHANGE: Nodes don't return the non-standard node.start & node.end anymore
@MichaelDeBoey
Copy link
Contributor Author

@ljharb All green, so this one can be merged too 🙂

@ljharb ljharb changed the title feat: Don't return node.start & node.end [Breaking]: Don't return node.start & node.end Jun 12, 2020
@ljharb
Copy link
Member

ljharb commented Jun 12, 2020

Thanks - it's good to have this queued up. However, I don't think we should rush to make a breaking change unnecessarily.

@MichaelDeBoey MichaelDeBoey force-pushed the remove-start-end branch 2 times, most recently from 765dc59 to b76766e Compare June 12, 2020 08:25
@ljharb ljharb merged commit 9de514f into jsx-eslint:master Oct 7, 2020
@MichaelDeBoey MichaelDeBoey deleted the remove-start-end branch October 7, 2020 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants