Skip to content

Add a release workflow that publishes to hex. Switch from deprecated… - #2

Merged
nathanmonteleone merged 15 commits into
masterfrom
publish_to_hex
Oct 21, 2026
Merged

nathanmonteleone merged 15 commits into
masterfrom
publish_to_hex

Conversation

@nathanmonteleone

Copy link
Copy Markdown

Also switches from deprecated setup-elixir to setup-beam

https://simplifi.atlassian.net/browse/DSP1-1032

nathanmonteleone and others added 11 commits October 15, 2026 14:24
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
ubuntu-22.04 runners only carry OTP 24+, so OTP 22 prebuilts are unavailable.
The hexpm/elixir Debian images ship every Elixir+OTP combination, so we run
each matrix entry in the matching container instead of using setup-beam to
install Erlang/Elixir on the host.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@coveralls

coveralls commented Oct 21, 2026 •

Copy link
Copy Markdown

Coverage Report for CI Build 219

Warning

No base build found for commit 3ce784e on master.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 92.588%

Details

  • Patch coverage: 2 uncovered changes across 1 file (2 of 4 lines covered, 50.0%).

Uncovered Changes

File Changed Covered %
lib/thrift/binary/framed/protocol_handler.ex 3 1 33.33%
Total (2 files) 4 2 50.0%

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 796
Covered Lines: 737
Line Coverage: 92.59%
Coverage Strength: 349.67 hits per line

💛 - Coveralls

OTP's ssl app no longer ships example certs in newer versions, so
Application.app_dir(:ssl, "examples/certs/etc/server") resolves to a
missing path. Bundle self-signed test certs in test/support/certs/ and
point get_certs/0 at them instead.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
nathanmonteleone and others added 2 commits October 21, 2026 14:44
The server-side cacertfile is only needed for mutual TLS (client cert
verification), which these tests don't use. Drop the file and the
option, leaving only certfile and keyfile.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

@kthomas-simplifi kthomas-simplifi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🐟 I'm willing to :rubber_stamp: anything for anyone who can get us out of the current cledos dependency hell trap.

@nathanmonteleone
nathanmonteleone merged commit ca40b00 into master Oct 21, 2026
6 checks passed
@LiruMouse
LiruMouse deleted the publish_to_hex branch October 21, 2026 20:47
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.

3 participants