search

rescript-association / reanalyze

Experimental analyses for ReScript and OCaml: globally dead values/types, exception analysis, and termination analysis.
MIT License
277 stars 20 forks source link

Interface of sub module are not seen used #186

Closed florentbarriol closed 1 year ago

florentbarriol commented 1 year ago

Hello :wave: , We use renalyze in our CI at Arte TV. And we face an issue with interface (.resi) of submodule. They do not seem to be seen by the analyze even if they are well used.

I create an example repository to illustrate the issue : https://github.com/florentbarriol/issue-interface-submodule

We find a workaround for now. We use inline interface and not .resi files. You will find an example in the repository.

Hope, you will find a solution because we love the projet. It's very helpful :heart:

cristianoc commented 1 year ago

@florentbarriol the analysis has moved inside the editor extension. The fix is here: https://github.com/rescript-lang/rescript-vscode/pull/650

cristianoc commented 1 year ago

The executable used by the editor extension has an option reanalyze. This is only used to test the extension at the moment, but could be used in CI as well as long as you get the extension's executable there. E.g. on my machine:

~/.vscode/extensions/chenglou92.rescript-vscode-1.9.35/server/analysis_binaries/darwinarm64/rescript-editor-analysis.exe reanalyze -help
florentbarriol commented 1 year ago

Thanks for your work !