Skip to content

Comments

Fix keyword argument 2.7#71

Merged
henriquegasques merged 2 commits intorubyritas:masterfrom
bodyshopbidsdotcom:fix-keyword-argument-2.7
Oct 16, 2023
Merged

Fix keyword argument 2.7#71
henriquegasques merged 2 commits intorubyritas:masterfrom
bodyshopbidsdotcom:fix-keyword-argument-2.7

Conversation

@lorman
Copy link
Contributor

@lorman lorman commented Oct 13, 2023

Noticed that there was still one place where the Ruby 2.7 keyword argument deprecation was being raised.

Running bundle exec rspec spec/ach/parse_spec.rb no longer raises deprecation errors.

Also noticed that the specs were failing and fixed that point of failure.

Copy link
Collaborator

@henriquegasques henriquegasques left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@henriquegasques henriquegasques merged commit 5fd38aa into rubyritas:master Oct 16, 2023
@lorman
Copy link
Contributor Author

lorman commented Oct 16, 2023

@henriquegasques can you please publish a new version of the gem when you get some time? Also, TY for accepting the contribution so quickly.

@henriquegasques
Copy link
Collaborator

@lorman NP, thank you for the contribution. Just released 0.6.3

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