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

Fix Transaction#params= deprecation warning #1110

Merged
merged 1 commit into from
Jun 24, 2024

Conversation

tombruijn
Copy link
Member

After PR #1109 our Ruby gem logs deprecation warnings about our internal Transaction#params= usage. Update it to Transaction#set_params to remove the deprecation messages.

[skip review]

@tombruijn tombruijn added the bug label Jun 24, 2024
@tombruijn tombruijn self-assigned this Jun 24, 2024
After PR #1109 our Ruby gem logs deprecation warnings about our internal
`Transaction#params=` usage. Update it to `Transaction#set_params` to
remove the deprecation messages.
@tombruijn tombruijn force-pushed the update-deprecated-params-usage branch from 003e1f9 to f5dddbf Compare June 24, 2024 12:55
@tombruijn tombruijn changed the base branch from main to hanami-spec-refactor June 24, 2024 13:05
@tombruijn tombruijn changed the base branch from hanami-spec-refactor to main June 24, 2024 13:05
@tombruijn tombruijn merged commit b65d667 into main Jun 24, 2024
16 checks passed
@tombruijn tombruijn deleted the update-deprecated-params-usage branch July 10, 2024 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant