8000 Escapes variable name in binary ser/de by nanodirijabl · Pull Request #3 · valitydev/elixir-thrift · GitHub
[go: up one dir, main page]

Skip to content

Escapes variable name in binary ser/de#3

Open
nanodirijabl wants to merge 13 commits intomasterfrom
ft/subst-reserved-vars
Open

Escapes variable name in binary ser/de#3
nanodirijabl wants to merge 13 commits intomasterfrom
ft/subst-reserved-vars

Conversation

@nanodirijabl
Copy link

No description provided.

pguillory and others added 5 commits December 16, 2022 13:32
Tests on other PRs are failing with the following error:

    Error: Requested Erlang/OTP version (23) not found in version list (should you be using option 'version-type': 'strict'?)

I tried changing the Ubuntu version used in the CI environment based on
Googling for ideas, and it seems to get the tests running.
Starting with Ranch 2.0, you get a warning when mixing Ranch and TCP socket
options in a single keyword list. We only support the old syntax, so you'll get
warnings at startup, and presumably that syntax will stop working at some
point.

This diff adds support for the new syntax. It does so in a backwards compatible
way, so you can upgrade Ranch and your option syntax either before or after
upgrading elixir-thrift.
@nanodirijabl nanodirijabl marked this pull request as draft May 7, 2024 11:27
@nanodirijabl nanodirijabl requested a review from ndiezel0 May 16, 2024 09:03
@nanodirijabl nanodirijabl marked this pull request as ready for review May 16, 2024 09:03
@nanodirijabl nanodirijabl requested a review from a team May 16, 2024 09:04
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

0