Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support Ruby 2.5 & 2.6 #6

Merged
merged 1 commit into from
May 3, 2022
Merged

Conversation

waiting-for-dev
Copy link
Contributor

Omnes was born as an extraction from
Solidus and it's used there to
provide pub/sub support. Current Solidus still supports Ruby 2.5 & 2.6.
We'll deprecate here as soon as they're deprecated on Solidus (which
should happen in next major).

An edge case that won't work on those versions is passing a serialized
payload when publishing an unstructured event, as it'll require the
kwargs keys to be Symbols. That's something not used on Solidus, so we
don't make any changes to support it. For those cases, the workaround is
publishing an event instance with the #payload method.

@waiting-for-dev waiting-for-dev force-pushed the waiting-for-dev/old_ruby branch 2 times, most recently from 380a0c7 to 938569a Compare May 3, 2022 04:48
Omnes was born as an extraction from
[Solidus](https://github.com/solidusio/solidus) and it's used there to
provide pub/sub support. Current Solidus still supports Ruby 2.5 & 2.6.
We'll deprecate here as soon as they're deprecated on Solidus (which
should happen in next major).

An edge case that won't work on those versions is passing a serialized
payload when publishing an unstructured event, as it'll require the
kwargs keys to be Symbols. That's something not used on Solidus, so we
don't make any changes to support it. For those cases, the workaround is
publishing an event instance with the `#payload` method.
@waiting-for-dev waiting-for-dev merged commit c7051b7 into main May 3, 2022
@waiting-for-dev waiting-for-dev deleted the waiting-for-dev/old_ruby branch May 3, 2022 04:51
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.

1 participant