Remove JSX/React eslint plugins.
What does this MR do?
Are there points in the code the reviewer needs to double check?
Why was this MR needed?
Screenshots (if relevant)
Does this MR meet the acceptance criteria?
-
Changelog entry added -
Documentation created/updated -
API support added - Tests
-
Added for this feature/bug -
All builds are passing
-
-
Conform by the merge request performance guides -
Conform by the style guides -
Branch has no merge conflicts with master
(if it does - rebase it please) -
Squashed related commits together
What are the relevant issue numbers?
Merge request reports
Activity
Added 1 commit:
-
59b37e1b - Change airbnb eslint config package to
eslint-config-airbnb-base
and update plugins.
-
59b37e1b - Change airbnb eslint config package to
Added 19 commits:
-
af555f91...b3616e30 - 18 commits from branch
master
- 90c3aa71 - Remove JSX/React eslint plugins.
-
af555f91...b3616e30 - 18 commits from branch
- Resolved by Luke "Jared" Bennett
Reassigned to @ClemMakesApps
@lbennett can you fix the code so that it obeys the updated
comma-dangle
rule instead of just disabling it, now that we know the reason?I think we can either put them all in one line or add a dangling comma
Reassigned to @lbennett
Reassigned to @ClemMakesApps
LGTM, @fatihacet can you review and merge?
Reassigned to @fatihacet
@lbennett Please add CHANGELOG.
Reassigned to @lbennett
@fatihacet Do we want a CHANGELOG for something like this? :o
Marked the task Conform by the merge request performance guides as completed
Marked the task Conform by the style guides as completed
Marked the task Squashed related commits together as completed