Skip to content

ESLint: Use babel-eslint parser to allow decorators and field initialization #2158

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

Merged
merged 1 commit into from
Jan 27, 2020

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Jan 26, 2020

This is needed to convert the app/router.js file to an ES6 class (see #2149 (comment)), and now the default in Ember.js apps (see https://github.com/ember-cli/ember-new-output/blob/v3.15.2/.eslintrc.js).

r? @locks

@locks
Copy link
Contributor

locks commented Jan 27, 2020

@bors r+

@bors
Copy link
Contributor

bors commented Jan 27, 2020

📌 Commit 020ba6c has been approved by locks

@bors
Copy link
Contributor

bors commented Jan 27, 2020

⌛ Testing commit 020ba6c with merge 7f31940...

bors added a commit that referenced this pull request Jan 27, 2020
ESLint: Use `babel-eslint` parser to allow decorators and field initialization

This is needed to convert the `app/router.js` file to an ES6 class (see #2149 (comment)), and now the default in Ember.js apps (see https://github.com/ember-cli/ember-new-output/blob/v3.15.2/.eslintrc.js).

r? @locks
@bors
Copy link
Contributor

bors commented Jan 27, 2020

☀️ Test successful - checks-travis
Approved by: locks
Pushing 7f31940 to master...

@bors bors merged commit 020ba6c into rust-lang:master Jan 27, 2020
@Turbo87 Turbo87 deleted the eslint branch March 30, 2020 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants