Skip to content

Add atomvm dialyzer provider#48

Merged
UncleGrumpy merged 1 commit into
atomvm:masterfrom
UncleGrumpy:dialyze
Jun 16, 2025
Merged

Add atomvm dialyzer provider#48
UncleGrumpy merged 1 commit into
atomvm:masterfrom
UncleGrumpy:dialyze

Conversation

@UncleGrumpy

@UncleGrumpy UncleGrumpy commented Jun 16, 2025

Copy link
Copy Markdown
Collaborator

Adds a new dialyzer task provider to simplify running dialyzer on AtomVM applications.

@UncleGrumpy

Copy link
Copy Markdown
Collaborator Author

This will need #47 before the OTP 25 test will pass.

@UncleGrumpy
UncleGrumpy requested a review from pguyot June 16, 2025 05:07
@UncleGrumpy
UncleGrumpy force-pushed the dialyze branch 3 times, most recently from 226040d to 1164f4b Compare June 16, 2025 05:46

@pguyot pguyot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for this.

I would rather call the task dialyzer so we would have:

rebar3 dialyzer (for Erlang/OTP)
rebar3 atomvm dialyzer (for AtomVM)

Comment thread src/atomvm_dialyzer_provider.erl Outdated
Comment thread src/atomvm_dialyzer_provider.erl
@UncleGrumpy
UncleGrumpy force-pushed the dialyze branch 3 times, most recently from dd76dd7 to 56bad53 Compare June 16, 2025 06:10
@UncleGrumpy

Copy link
Copy Markdown
Collaborator Author

Thank you for this.

I would rather call the task dialyzer so we would have:

rebar3 dialyzer (for Erlang/OTP) rebar3 atomvm dialyzer (for AtomVM)

Done. I started off that way, but because some options are different I used a different name, but the namespace should suffice to let users know it is not the identical task.

@UncleGrumpy
UncleGrumpy force-pushed the dialyze branch 2 times, most recently from ad2e8cd to 0c16a28 Compare June 16, 2025 06:46
Adds a new `dialyzer` task provider to simplify running dialyzer on AtomVM applications.

Signed-off-by: Winford <winford@object.stream>
@UncleGrumpy UncleGrumpy changed the title Add atomvm dialyze provider Add atomvm dialyzer provider Jun 16, 2025
@UncleGrumpy
UncleGrumpy merged commit 54a9b71 into atomvm:master Jun 16, 2025
6 checks passed
@UncleGrumpy
UncleGrumpy deleted the dialyze branch February 16, 2026 21:44
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.

2 participants