Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix dialyzer test when running locally #271

Merged
merged 2 commits into from
May 29, 2020
Merged

Fix dialyzer test when running locally #271

merged 2 commits into from
May 29, 2020

Conversation

axelson
Copy link
Member

@axelson axelson commented May 29, 2020

I'm testing running locally on arch linux. It appear that for me locally for the "reports diagnostics then clears them once problems are fixed" test I get the output I get the message:

[ElixirLS Dialyzer] Analyzing 3 modules: [A, B, :erlang]

Instead of:

[ElixirLS Dialyzer] Analyzing 2 modules: [A, B]

Adding a function definition in module C causes :erlang to not be detected as needing to be analyzed.

I'm testing running locally on arch linux. It appear that for me locally
for the "reports diagnostics then clears them once problems are fixed"
test I get the output I get the message:

    [ElixirLS Dialyzer] Analyzing 3 modules: [A, B, :erlang]

Instead of:

    [ElixirLS Dialyzer] Analyzing 2 modules: [A, B]

Adding a function definition in module C causes :erlang to not be
detected as needing to be analyzed.
@axelson axelson merged commit 269e78d into elixir-lsp:master May 29, 2020
@axelson axelson deleted the fix-dialyzer-test-local branch May 29, 2020 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant