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

Elixir checker accept optional environment #1940

Closed
wants to merge 1 commit into from
Closed

Elixir checker accept optional environment #1940

wants to merge 1 commit into from

Conversation

tsubery
Copy link

@tsubery tsubery commented Dec 30, 2016

The solution proposed here wasn't working for. elixirc reported errors on many files that mix wasn't. It must have something to do with build environments.

With this patch we can set let g:syntastic_elixir_compile_env = { 'MIX_ENV': 'test' } and
workaround this issue by building in a different environment.

@lcd047
Copy link
Collaborator

lcd047 commented Jan 2, 2017

The elixir checker is already broken beyond fixing, mainly because error messages produced by Elixir are way too hard to parse with errorformat. So this checker is pretty high on the list of things to remove in a future release of syntastic.

In the mean time, you could achieve what you want without changes to the code, by using a script wrapper that sets whatever environment variables you want. Thus, sorry, but no.

@lcd047 lcd047 closed this Jan 2, 2017
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.

None yet

2 participants