Skip to content
This repository was archived by the owner on Sep 10, 2021. It is now read-only.

Linted codebase to meet StandardJS formatting #34

Merged
merged 4 commits into from
Sep 19, 2017
Merged

Linted codebase to meet StandardJS formatting #34

merged 4 commits into from
Sep 19, 2017

Conversation

ramfox
Copy link
Member

@ramfox ramfox commented Sep 18, 2017

  • Switched to formatting standard to Standard JS
  • Linted the codebase using ESLint. Added ESLint, and added necessary eslint React, Babel, and StandardJS packages to the devDependencies in package.json
  • Added standard to devDependencies in package.json -> standard is a command line tool that allows you to view linting errors from the terminal. You can also use it to batch-fix basic linting errors
  • In package.json -> Added specific directories and files for standard to ignore (files added were either trouble spots or unnecessary to lint)

Linting makes code better :) It adds to the code's readability and can help find bugs.

@b5
Copy link
Member

b5 commented Sep 19, 2017

This looks great! We had a quick chat about adding more commits per PR, dat's it.

@b5 b5 self-requested a review September 19, 2017 15:37
@ramfox ramfox merged commit 0d18dda into master Sep 19, 2017
@ramfox ramfox mentioned this pull request Sep 19, 2017
10 tasks
@b5 b5 deleted the lint branch September 21, 2017 22:59
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