project-nv / night-vision

Highly customizable charting library, created for professional traders
https://nightvision.dev
MIT License
242 stars 57 forks source link

Integrate eslint #94

Closed mariusvigariu closed 1 year ago

mariusvigariu commented 1 year ago

Description

As a dev I would like to have eslint integrated to make the code more readable

Suggested solution

Integrate eslint, keep defaults and add the following rules:

cat .eslintrc
{
  "rules": {
    "comma-dangle": ["error", "always-multiline"]
  }
}

Alternative

No response

Additional context

No response

Validations

C451 commented 1 year ago

Add eslint for checking commas? hmmm

mariusvigariu commented 1 year ago

Thats just an example for the rc file, it does not mean its only checking commas :)

I strongly suggest integrating eslint as it will make the code much more readable, thus easier to maintain and extend in the future