Skip to content

Conversation

@lucasmenezesds
Copy link

@lucasmenezesds lucasmenezesds commented Mar 30, 2024

Describe the change

What does this Pull Request do?

The PR address the bug from the issue #197

Why are we doing this?

Any related context as to why is this is a desirable change.

To fix the bug of not being able to define custom messages from key arguments

Benefits

How will the library improve?

Fix the issue of not being able to define custom messages from key arguments, (previously was working only be block usage)

Drawbacks

Possible drawbacks applying this change.

Requirements

  • Tests written & passing locally?
  • Code style checked?
  • Rebased with master branch?
  • Documentation updated?
  • Changelog updated?

@lucasmenezesds lucasmenezesds force-pushed the allow-msg-as-key-arg branch 3 times, most recently from cf563e4 to 6f901fa Compare March 30, 2024 09:17
@lucasmenezesds lucasmenezesds marked this pull request as ready for review March 30, 2024 09:17
@lucasmenezesds lucasmenezesds force-pushed the allow-msg-as-key-arg branch 2 times, most recently from c73f44c to c97541c Compare March 30, 2024 09:25
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