Skip to content
This repository has been archived by the owner on Nov 10, 2022. It is now read-only.

Update: remove parser demo #664

Merged
merged 1 commit into from Dec 6, 2019
Merged

Update: remove parser demo #664

merged 1 commit into from Dec 6, 2019

Conversation

kaicataldo
Copy link
Member

@kaicataldo kaicataldo commented Dec 5, 2019 •
edited

Refs #652

This PR removes the parser demo.

Two follow-up tasks that I'll take care once we land this:

  1. Remove the bundling of Espree in eslint/eslint.
  2. Replace instances of referring users to the parser with links to AST Explorer instead.

Screen Shot 2019-12-04 at 10 30 31 PM

@eslint-deprecated eslint-deprecated bot added the triage An ESLint team member will look at this issue soon label Dec 5, 2019
@kaicataldo kaicataldo added enhancement This change enhances an existing feature of the website and removed triage An ESLint team member will look at this issue soon labels Dec 5, 2019
@kaicataldo kaicataldo changed the title Enhancement: remove parser demo Update: remove parser demo Dec 5, 2019
@kaicataldo kaicataldo added the demo Relates to the ESLint demo page (https://eslint.org/demo) label Dec 5, 2019
@kaicataldo kaicataldo mentioned this pull request Dec 5, 2019
Merged
@kaicataldo kaicataldo added the accepted There is consensus among the team that this change meets the criteria for inclusion label Dec 5, 2019
@kaicataldo
Copy link
Member Author

eslint/eslint PR here.

@kaicataldo kaicataldo mentioned this pull request Dec 5, 2019
Merged
@kaicataldo kaicataldo merged commit b01cf98 into master Dec 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@platinumazure platinumazure platinumazure approved these changes

@ilyavolodin ilyavolodin ilyavolodin approved these changes

@btmills btmills btmills approved these changes

Assignees
No one assigned
Labels
accepted There is consensus among the team that this change meets the criteria for inclusion demo Relates to the ESLint demo page (https://eslint.org/demo) enhancement This change enhances an existing feature of the website
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@kaicataldo @platinumazure @ilyavolodin @btmills