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

Maintenance #60

Merged
merged 14 commits into from
Feb 19, 2019
Merged

Maintenance #60

merged 14 commits into from
Feb 19, 2019

Conversation

Arcanemagus
Copy link
Member

Several small changes bringing this package up to a better standing:

  • Migrate to a CircleCI 2.0 config
  • Fix usage of uuid4
  • Adds some basic specs
  • Fix function to write socket data broken in Update linting configuration #55
  • Specify as able to lint on changes

Fixes #2.
Fixes #48.
Fixes #58.

Update to a new CircleCI 2.0 configuration based on the 
arcanemagus/atom-docker-ci image.
The previous specs appear to have been some sort of custom interface
that no longer exists. Implement new specs that test the basic
functionality of the linter.

Note that it seems `Lint.jl` only ever returns a column range of 0 to 80
currently.

Fixes #2.
This needs to inherit the `this` from the calling socket connection.
@Arcanemagus Arcanemagus self-assigned this Feb 19, 2019
This was switched in #41, but no reasoning was given. Since it appears 
the server handles this we should re-enable it.

Fixes #48.
The v0.5.2, although it is the version the current Lint.jl is designed 
for, is unable to contact GitHub to download the package. Switch to 
v0.6.4 in order to be able to install the package.
This should automatically install for live environments, however it is 
too slow to re-initialize for each spec.
Fix the `console.debug` statements to actually work, and only print them 
when Atom is in developer mode.
Julia is _very_ slow to bring in Lint.jl when starting up, increase the 
spec timeout to 90 seconds from 10 seconds in order to accommodate that.
@Arcanemagus Arcanemagus merged commit 07503f1 into master Feb 19, 2019
@Arcanemagus Arcanemagus deleted the arcanemagus/maintenance branch February 19, 2019 20:34
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.

1 participant