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

Rewrite in JS #57

Merged
merged 2 commits into from
Aug 27, 2017
Merged

Rewrite in JS #57

merged 2 commits into from
Aug 27, 2017

Conversation

Arcanemagus
Copy link
Member

@Arcanemagus Arcanemagus commented Aug 26, 2017

Rewrite the package in JavaScript, including:

  • Checking whether clojure.jar isn't found
  • Update to Linter v2 API
  • Deferral of dependency loading till needed
  • Moving configuration to package.json
  • Observing the settings so Atom doesn't need to determine them each time
  • Adding ESLint and configuration

@Arcanemagus
Copy link
Member Author

@ehteqx Would you mind testing this branch out? I don't have an environment setup to test this, nor files to test on 😉.

@Arcanemagus
Copy link
Member Author

Got an environment setup and tested this. Fixed a few bugs in the initial implementation and added specs.

Rewrite the package in JavaScript, including:
* Checking whether `clojure.jar` isn't found
* Update to Linter v2 API
* Deferral of dependency loading till needed
* Moving configuration to package.json
* Observing the settings so Atom doesn't need to determine them each
time
* Adding ESLint and configuration
Add specs testing the functionality of the package.
@Arcanemagus Arcanemagus merged commit b72cac7 into master Aug 27, 2017
@Arcanemagus Arcanemagus deleted the arcanemagus/js-rewrite branch August 27, 2017 22:31
@Arcanemagus Arcanemagus mentioned this pull request Aug 27, 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