Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

Dependency updates #4

Merged
merged 3 commits into from
Feb 2, 2019
Merged

Dependency updates #4

merged 3 commits into from
Feb 2, 2019

Conversation

Windvis
Copy link
Contributor

@Windvis Windvis commented Feb 1, 2019

Resolves #1 and #2.

Sam Van Campenhout added 2 commits February 1, 2019 15:29
Without the user having this installed, it is possible to trigger 'Definition for rule 'rule-name' was not found'
BREAKING CHANGE: Since `@bagaar/eslint-config-ember` does not override any rules of `@bagaar/eslint-config`, extending has no real benefit besides not having to include `@bagaar/eslint-config` inside a project`s extends.
Removing the extend also has the benefit that we can switch out the ESLint specific config without having to change this library.
Remove references to `@bagaar/eslint-config-ember` and add a note about the eslint-plugin-ember dependency.
@bertdeblock bertdeblock merged commit 1ec3acb into master Feb 2, 2019
Windvis added a commit that referenced this pull request Feb 4, 2019
* chore: Add eslint-plugin-ember as a peer dependency

Without the user having this installed, it is possible to trigger `Definition for rule 'rule-name' was not found`

* chore: Don't extend `@bagaar/eslint-config`

BREAKING CHANGE: Since `@bagaar/eslint-config-ember` does not override any rules of `@bagaar/eslint-config`, extending has no real benefit besides not having to include `@bagaar/eslint-config` inside a project's extends.
Removing the extend also has the benefit that we can switch out the ESLint specific config without having to change this library.

* chore: Update the README

Remove references to `@bagaar/eslint-config-ember` and add a note about the eslint-plugin-ember dependency.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants