This repository has been archived by the owner on Aug 7, 2023. It is now read-only.
Provide a default PATH for finding shellcheck #7
Labels
Comments
|
Actually copied Rather than Which gives me the same error @alex commented issued. Anyhow, I needed to fix it manually |
|
should be fixed by #21 . Default is just 'shellcheck' which needs to be available by $PATH. |
lexicalunit
added a commit
to lexicalunit/linter-shellcheck
that referenced
this issue
Dec 4, 2015
On OS X electron frequently does not seem to pick up the PATH setting correctly when launching Atom from a GUI, for example the Dock. This means that shellcheck can not be found. One workaround is to specify the absolute path to shellcheck, but this can be done automatically. This provides a much better experience for users on OS X. This should also resolve issue AtomLinter#7 for many users. - Adds `enableFixPath` option. - Adds documentation for option to README. - Adds observer for option, once turned on it can't be reversed. - Defaults `enableFixPath` to true for 'darwin` (OS X) platform. - Provides user documentation for option, indicating its use in OS X.
Closed
lexicalunit
added a commit
to lexicalunit/linter-shellcheck
that referenced
this issue
Dec 4, 2015
On OS X electron frequently does not seem to pick up the PATH setting correctly when launching Atom from a GUI, for example the Dock. This means that shellcheck can not be found. One workaround is to specify the absolute path to shellcheck, but this can be done automatically. This provides a much better experience for users on OS X. This should also resolve issue AtomLinter#7 for many users. - Adds `enableFixPath` option. - Adds documentation for option to README. - Adds observer for option, once turned on it can't be reversed. - Defaults `enableFixPath` to true for 'darwin` (OS X) platform. - Provides user documentation for option, indicating its use in OS X. - Adds optional `fix-path` dependency (avoid issues with non-OS X OS's).
lexicalunit
added a commit
to lexicalunit/linter-shellcheck
that referenced
this issue
Dec 4, 2015
On OS X electron frequently does not seem to pick up the PATH setting correctly when launching Atom from a GUI, for example the Dock. This means that shellcheck can not be found. One workaround is to specify the absolute path to shellcheck, but this can be done automatically. This provides a much better experience for users on OS X. This should also resolve issue AtomLinter#7 for many users. - Adds `enableFixPath` option. - Adds documentation for option to README. - Adds observer for option, once turned on it can't be reversed. - Defaults `enableFixPath` to true for 'darwin` (OS X) platform. - Provides user documentation for option, indicating its use in OS X. - Adds `fix-path` dependency.
|
Marking this as closed, the default is now |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Right now the default is empty, which results in ENOENT the first time you load up a shell script, it would be better if a default of some sort was provided.
The text was updated successfully, but these errors were encountered: