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: PDF Filler Spec #11920

Merged
merged 0 commits into from
Mar 2, 2023
Merged

Ruby 3 KwArgs: PDF Filler Spec #11920

merged 0 commits into from
Mar 2, 2023

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 PdfFill::Filler spec.

@ericboehs ericboehs requested review from a team as code owners February 28, 2023 15:47
@ericboehs ericboehs enabled auto-merge (squash) February 28, 2023 15:47
@va-vfs-bot va-vfs-bot temporarily deployed to eb-kwargs-pdf-filler-spec/main/main February 28, 2023 15:50 Inactive
holdenhinkle
holdenhinkle previously approved these changes Feb 28, 2023
@ericboehs ericboehs force-pushed the eb-kwargs-pdf-filler-spec branch from 263e473 to 8168748 Compare February 28, 2023 18:51
@va-vfs-bot va-vfs-bot temporarily deployed to eb-kwargs-pdf-filler-spec/main/main February 28, 2023 19:04 Inactive
@ericboehs ericboehs force-pushed the eb-kwargs-pdf-filler-spec branch from 8168748 to 409d232 Compare March 1, 2023 16:06
@ericboehs ericboehs disabled auto-merge March 2, 2023 20:46
@ericboehs ericboehs merged commit 2d82cfc into master Mar 2, 2023
@ericboehs ericboehs deleted the eb-kwargs-pdf-filler-spec branch March 2, 2023 20:47
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