Skip to content

fix: passing the adapter_opts properly#431

Merged
sleipnir merged 5 commits intoelixir-grpc:masterfrom
xinguanz:fix/start-server-keyword-adapter-opts
Aug 8, 2025
Merged

fix: passing the adapter_opts properly#431
sleipnir merged 5 commits intoelixir-grpc:masterfrom
xinguanz:fix/start-server-keyword-adapter-opts

Conversation

@xinguanz
Copy link
Contributor

@xinguanz xinguanz commented Aug 4, 2025

The adapter_opts keyword list encapsulates the :ip and :net options.

Copy link
Collaborator

@sleipnir sleipnir left a comment

Choose a reason for hiding this comment

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

Just a few minor comments and questions

Copy link
Collaborator

@sleipnir sleipnir left a comment

Choose a reason for hiding this comment

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

LGTM

@sleipnir
Copy link
Collaborator

sleipnir commented Aug 5, 2025

@XinGuanTubi please check the formatting

@xinguanz
Copy link
Contributor Author

xinguanz commented Aug 6, 2025

@sleipnir I've formatted the file

@sleipnir
Copy link
Collaborator

sleipnir commented Aug 7, 2025

@sleipnir I've formatted the file

Tks!

@sleipnir
Copy link
Collaborator

sleipnir commented Aug 7, 2025

Hi @XinGuanTubi some tests failed! https://github.com/elixir-grpc/grpc/actions/runs/16768441575/job/47612192934
You can take a look?

@xinguanz
Copy link
Contributor Author

xinguanz commented Aug 8, 2025

@sleipnir I've fixed the tests that have GRPC.Server.start by replacing opts with adapter_opts.
I think it's what the version v0.10 want us to do so.
All tests should be passing now.

@sleipnir sleipnir merged commit faee2c1 into elixir-grpc:master Aug 8, 2025
19 of 21 checks passed
@sleipnir
Copy link
Collaborator

sleipnir commented Aug 8, 2025

@XinGuanTubi Thank you for your contribution

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