Skip to content

feat: suport named parameters#3245

Open
yenfryherrerafeliz wants to merge 1 commit intoaws:masterfrom
yenfryherrerafeliz:feat_client_support_named_parameters
Open

feat: suport named parameters#3245
yenfryherrerafeliz wants to merge 1 commit intoaws:masterfrom
yenfryherrerafeliz:feat_client_support_named_parameters

Conversation

@yenfryherrerafeliz
Copy link
Contributor

Add support to named parameters by evaluating if the named parameter key is present in the arguments given to __call in AwsClientTrait.

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Add support to named parameters by evaluating if the named parameter key is present in the arguments given to __call in AwsClientTrait.
Copy link
Member

@stobrien89 stobrien89 left a comment

Choose a reason for hiding this comment

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

One minor comment that doesn't need a fix, but a changelog entry should be added under the enhancement category.

Copy link
Member

@stobrien89 stobrien89 Feb 16, 2026

Choose a reason for hiding this comment

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

pedantic nit, but test name should be testSupportsNamedArgs

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