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

Rewrite in ES2017 #18

Merged
merged 5 commits into from
Mar 15, 2017
Merged

Rewrite in ES2017 #18

merged 5 commits into from
Mar 15, 2017

Conversation

Arcanemagus
Copy link
Member

Rewrite the linter in ES2017, including:

  • Observe all the settings instead of retrieving them every time
  • Fix the regex to pull in the info as well
  • Don't lint on every modification of a file, only on save
  • Remove the no-color option, it could only have potentially caused problems
  • Use atom-linter to execute the program
  • Add the [INFO] lines as traces on the regular messages

Also includes some basic specs and general repository maintenance.

Add an ESLint configuration matching most of the rest of the projects in
this organization.
Some basic tests validating that it works properly for a "valid"
project, and can report errors for an invalid project.
Rewrite the linter in ES2017, including:
* Observe all the settings instead of retrieving them every time
* Fix the regex to pull in the info as well
* Don't lint on every modification of a file, only on save
* Remove the no-color option, it could only have potentially caused problems
* Use `atom-linter` to execute the program
* Add the [INFO] lines as traces on the regular messages
@Arcanemagus Arcanemagus merged commit 2b9d384 into master Mar 15, 2017
@Arcanemagus Arcanemagus deleted the arcanemagus/es2017-rewrite branch March 15, 2017 22:25
@Arcanemagus Arcanemagus mentioned this pull request Mar 15, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant