Skip to content

Support OTP/28 in CI#80

Merged
josevalim merged 1 commit intobeam-telemetry:mainfrom
myfloss:support-otp-28-in-ci
Oct 17, 2025
Merged

Support OTP/28 in CI#80
josevalim merged 1 commit intobeam-telemetry:mainfrom
myfloss:support-otp-28-in-ci

Conversation

@kianmeng
Copy link
Contributor

Also bump actions/checkout, bump rebar3, resolve dialyzer error, and add missing dialyzer specs.

Also bump actions/checkout, bump rebar3, resolve dialyzer error, and add
missing dialyzer specs.

-spec make_measurement(measurement()) -> measurement() | no_return().
-spec make_measurement(measurement()) -> measurement() | error.
make_measurement(Measurement = {M, F, A}) ->
Copy link
Contributor Author

Choose a reason for hiding this comment

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

/tmp/telemetry_poller/src/telemetry_poller.erl:470:50: The test 
          {atom(), atom(), [any()]} =/= 
          'error' can never evaluate to 'false'

===> Warnings written to _build/default/28.1.dialyzer_warnings

Comment on lines +17 to +32
include:
- otp_version: "24"
rebar3_version: "3.21.0"
os: ubuntu-latest
- otp_version: "25"
rebar3_version: "3.21.0"
os: ubuntu-latest
- otp_version: "26"
rebar3_version: "3.25.1"
os: ubuntu-latest
- otp_version: "27"
rebar3_version: "3.25.1"
os: ubuntu-latest
- otp_version: "28"
rebar3_version: "3.25.1"
os: ubuntu-latest
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@josevalim josevalim merged commit f93559c into beam-telemetry:main Oct 17, 2025
5 checks passed
@josevalim
Copy link
Contributor

💚 💙 💜 💛 ❤️

@kianmeng
Copy link
Contributor Author

🥳 🥳 🥳 🥳 🥳

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants