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

fix Ruby 2.7 deprecation warnings #438

Merged
merged 1 commit into from
Mar 30, 2020
Merged

Conversation

breunigs
Copy link
Contributor

Example:

…/gems/kubeclient-4.6.0/lib/kubeclient.rb:27: warning: Using the last argument as keyword parameters is deprecated; maybe ** should be added to the call
…/gems/kubeclient-4.6.0/lib/kubeclient/common.rb:61: warning: The called method `initialize_client' is defined here

I have not setup the CI locally, so let's see if this passes.

Example:
```
…/gems/kubeclient-4.6.0/lib/kubeclient.rb:27: warning: Using the last argument as keyword parameters is deprecated; maybe ** should be added to the call
…/gems/kubeclient-4.6.0/lib/kubeclient/common.rb:61: warning: The called method `initialize_client' is defined here
```
@breunigs
Copy link
Contributor Author

Fixes #431

@cben
Copy link
Collaborator

cben commented Mar 30, 2020

Thanks 🙏!
Let me just confirm whether this method has good tests...

@cben
Copy link
Collaborator

cben commented Mar 30, 2020

LGTM 💯

@cben cben merged commit 4b79df3 into ManageIQ:master Mar 30, 2020
@breunigs breunigs deleted the ruby27-warnings branch March 31, 2020 11:07
@grosser
Copy link
Contributor

grosser commented May 27, 2020

still seeing this, can we please get a release ?

cben added a commit to cben/kubeclient that referenced this pull request Jun 14, 2020
@cben
Copy link
Collaborator

cben commented Jun 14, 2020

Sorry, had lots of pressure at work...
Released in https://rubygems.org/gems/kubeclient/versions/4.7.0 (#446).

@grosser
Copy link
Contributor

grosser commented Jun 14, 2020

thank you so much, sweet silence :D

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.

3 participants