Skip to content
This repository has been archived by the owner on Feb 10, 2024. It is now read-only.

Lint on change #1

Closed
elstgav opened this issue May 11, 2015 · 2 comments
Closed

Lint on change #1

elstgav opened this issue May 11, 2015 · 2 comments
Assignees

Comments

@elstgav
Copy link
Owner

elstgav commented May 11, 2015

SublimeLinter uses temp files to lint while you code. Problem is these temp files are created in a separate directory, so slim-lint can't do its automatic detection of the config file using the file path. We get around this by using the --config option with slim-lint. However, we can't supply the location of the rubocop config, so when slim-lint runs rubocop it resorts to its default configuration.

This leaves us two options:

  1. Lint the actual file after its saved. This is less than ideal, as it slows down the linting process, and SublimeLinter will actually freeze SublimeText while it does its thing. Very annoying and cumbersome.
  2. Modify slim-lint so that we can manually set the config path for rubocop.

For now I'll pursue option 2.

@elstgav elstgav added the bug label May 11, 2015
@elstgav elstgav self-assigned this May 11, 2015
elstgav added a commit that referenced this issue May 17, 2015
Until we can manually set rubocop's config location, we need to lint
on save so relative paths are correct.
@elstgav elstgav added enhancement and removed bug labels May 17, 2015
@elstgav elstgav changed the title Does not load rubocop configuration Lint on change May 17, 2015
@yvbeek
Copy link

yvbeek commented May 23, 2015

I noticed this issue, as soon as I correct one of the issues all linter errors are gone.
Great work, looking forward to this fix!

@elstgav
Copy link
Owner Author

elstgav commented May 31, 2015

Released in v2.0.0 (64e1acb)

@elstgav elstgav closed this as completed May 31, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants