Serubin / pulse-sms-web

The official web app for Pulse SMS - built on Vue.js.
https://pulsesms.app
Other
111 stars 44 forks source link

Fix and re-enable eslint #151

Closed Serubin closed 3 years ago

Serubin commented 3 years ago

Description

This PR fixes and re-enables eslint. This does not fix any files. We will need to go through bit by bit and fix files. Most fit pretty closely to Vue standard - I have a few non-standard styles that should be removed and are addressed in these codified style changes.

Rules can be found here: https://eslint.vuejs.org/rules/ this is using recommended rules.

Other notes

This also reintroduced the yarn.lock tracking - it was removed a while back, I'm not sure why.

Serubin commented 3 years ago

@mohsintahir @MapleSimon What's the process for getting a review?

mohsintahir commented 3 years ago

@Serubin thanks alot for this PR. Great help.