Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Remove confusing lint rules #168

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

rsexton404
Copy link

@rsexton404 rsexton404 commented Apr 10, 2019

Changes

  • Move lint rule on curly brace formatting that is incompatible with Prettier
  • Remove lint rules that require a newline at the end of the file and sorted proptypes

How To Test

  • Clone this repo and check out this branch
  • Run npm link inside the mobify-code-style
  • In any project that has mobify-code-style as a dependency, run npm link mobify-code-style
  • Open any source file in the project and try to trigger the lint errors
  • Run the linter: npm run lint or use an ESLint extension for your editor
  • You should not see lint erros for the formatting issues described in the changes section

TODOs:

  • +1
  • Updated README
  • Updated CHANGELOG

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

1 participant