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

Ruby 3 KwArgs: SignInController spec #11916

Merged
merged 0 commits into from
Feb 28, 2023
Merged

Conversation

ericboehs
Copy link
Contributor

Ruby 2.7 deprecated automatic casting of a hash passed as the last argument of a method to keyword arguments. You must now use actual keyword arguments (foo(bar: 'baz') instead of foo({bar: 'baz'})) or cast the hash to keyword args using the double splat (foo(**{bar: 'baz'})).

In preparation for upgrading to Ruby 3.0, we're implementing the new syntax for SignInController spec.

@ericboehs ericboehs changed the title Fix sign in controller spec (kwarg) Ruby 3 KwArgs: SignInController spec Feb 28, 2023
@ericboehs ericboehs force-pushed the eb-kwarg-signin-controler-spec branch from f8589ba to 2607ae4 Compare February 28, 2023 15:08
@ericboehs ericboehs marked this pull request as ready for review February 28, 2023 15:11
@ericboehs ericboehs requested a review from a team as a code owner February 28, 2023 15:11
@ericboehs ericboehs enabled auto-merge (squash) February 28, 2023 15:11
@ericboehs ericboehs disabled auto-merge February 28, 2023 17:35
@ericboehs ericboehs merged commit 395091a into master Feb 28, 2023
@ericboehs ericboehs deleted the eb-kwarg-signin-controler-spec branch February 28, 2023 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants