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

Update dependencies; add some new rules #9

Merged
merged 1 commit into from
Oct 23, 2019
Merged

Update dependencies; add some new rules #9

merged 1 commit into from
Oct 23, 2019

Conversation

edsrzf
Copy link
Collaborator

@edsrzf edsrzf commented Oct 23, 2019

This does a few things:

  • Updates dependencies, including some major version bumps (eslint, @typescript-eslint, standard)
  • Add eslint-plugin-react-hooks for React hook lint rules
  • Add eslint-plugin-jsx-a11y, which create-react-app also uses. Only a few rules enabled for now, but we should aim to get to at least the same configuration as create-react-app.
  • Make config dependencies non-peer dependencies, since it's now possible with eslint 6.
  • Add dev dependencies and a lint script to this repo to make it easier to test the validity of the configuration.

Tested configuration against Web Register and Monocle. Some new issues, but pretty valid ones.

This does a few things:
* Updates dependencies, including some major version bumps (eslint,
  @typescript-eslint)
* Add eslint-plugin-react-hooks for React hook lint rules
* Add eslint-plugin-jsx-a11y, which create-react-app also uses. Only
  a few rules enabled for now, but we should aim to get to at least the
  same configuration as create-react-app.
* Make config dependencies non-peer dependencies, since it's now
  possible with eslint 6.
* Add dev dependencies and a lint script to this repo to make it easier to test
  the validity of the configuration.

Tested configuration against Web Register and Monocle. Some new issues,
but pretty valid ones.
@edsrzf edsrzf requested review from aarr0n and N1ck October 23, 2019 19:10
Copy link

@N1ck N1ck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@edsrzf edsrzf merged commit da4f5d6 into master Oct 23, 2019
@edsrzf edsrzf deleted the update branch October 23, 2019 20:34
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.

2 participants