Skip to content

Conversation

@bjosv
Copy link
Collaborator

@bjosv bjosv commented Nov 20, 2025

Re-enable the unknown function or type warning that previously was disabled for handling OTP 26.

Additional apps are needed in the PLT and added via plt_extra_apps to handle the previously hidden warnings, like:

test/ered_client_tests.erl   Line 0: Unknown function eunit:test/1
./ssl.erl                    Line 927 Column 77: Unknown type public_key:oid/0

This PR also updates the CI:

  • Update CI to only run on OTP 27 and 28 with matching rebar3, and lift versions.
  • Add valkey-version 9.0.0 and 8.1.4 to the db-compatibility CI job.
  • The lift of rebar3 created problems in the release CI job, which now uses erl instead of rebar3 shell.
    rebar3 seems to have changed so its shell only runs in interactive mode, but Github Actions don't support TTYs and halts without running.

@bjosv bjosv requested a review from zuiderkwast November 20, 2025 21:20
Use a rebar3 versions that matches OTP 27 and 28

Signed-off-by: Björn Svensson <[email protected]>
rebar3 seems to have changed so its shell only runs in interactive mode,
which Github Actions TTY does not support and halts without running.

Signed-off-by: Björn Svensson <[email protected]>
Copy link
Collaborator

@zuiderkwast zuiderkwast left a comment

Choose a reason for hiding this comment

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

OK, then, since rebar3 as test dialyzer seems to work with the workaround, I suppose we can merge this as is.

Should we preserve the individual commits or squash them?

@bjosv bjosv changed the title Fix dialyzer warnings about unknown types and functions Re-enable 'unknown types and functions' warnings and update CI Nov 21, 2025
@bjosv
Copy link
Collaborator Author

bjosv commented Nov 21, 2025

Should we preserve the individual commits or squash them?

I have updated the PR header now. Squashing them is fine with me.

@bjosv bjosv merged commit c8486a9 into Ericsson:main Nov 21, 2025
14 of 15 checks passed
@bjosv bjosv deleted the fix-dialyzer branch November 21, 2025 09:46
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