WhatsApp / erlang-language-platform

Erlang Language Platform. LSP server and CLI.
https://whatsapp.github.io/erlang-language-platform/
Apache License 2.0
209 stars 16 forks source link

elp eqwalize-all finding no files to analyze #10

Closed tsloughter closed 8 months ago

tsloughter commented 8 months ago

Describe the bug

$ elp eqwalize-all --rebar                                                    ─╯
o⠁ Loading build info                                                                Loading build info                                                                 Loading applications      ████████████████████ 58/58                               Seeding database                                                                   Compiling dependencies                                                             Gathering modules         ████████████████████ 203/203                           No files to eqWAlize detected

To Reproduce**

Clone https://github.com/open-telemetry/opentelemetry-erlang and run rebar3 upgrade --all and rebar3 plugins upgrade --all (haven't upgraded the eqwalizer main yet because I'm trying to figure this out before bumping elp), then run elp eqwalize-all --rebar

Note the same happens for per-app runs:

$ elp eqwalize-app --rebar opentelemetry_api                                  
  Loading build info                                                                 Loading applications      ████████████████████ 58/58                               Seeding database                                                                   Compiling dependencies                                                           No files to eqWAlize detected

Expected behavior

Expected analysis of all applications in the project.

Actual behavior

No modules are analyzed

Context

alanz commented 8 months ago

There are two things affecting this.

alanz commented 8 months ago

And the latest release runs eqwalizer, and opts all files in for typechecking by default.

But it still has a problem with #12, where it is not able to resolve an include file for _build/default/lib/grpcbox/src/grpcbox_client.erl:25:10.

tsloughter commented 8 months ago

Thanks :+1:

alanz commented 8 months ago

I just checked current master (da0b80b04ab28e75ab1220398bfabc183d1f41fa) and it seems to work.

We will do a new release soon.

alanz commented 8 months ago

Please try again with today's release, and re-open if it is still a problem.