Skip to content

Removed some warnings#30

Closed
kachick wants to merge 3 commits into
ipaddress-gem:masterfrom
kachick:improve-remove_warnings
Closed

Removed some warnings#30
kachick wants to merge 3 commits into
ipaddress-gem:masterfrom
kachick:improve-remove_warnings

Conversation

@kachick
Copy link
Copy Markdown
Contributor

@kachick kachick commented Jan 25, 2013

They remove below ruby's warnings.
I have tested on MRI 1.9.3

  • warning: mismatched indentations
  • warning: (...) interpreted as grouped expression (in test)

It removes below ruby's warning.

* "mismatched indentations"
It removes below ruby's warning when running test.

* "(...) interpreted as grouped expression"
@smortex smortex mentioned this pull request Aug 30, 2017
@smortex
Copy link
Copy Markdown
Contributor

smortex commented Aug 30, 2017

Rebased and integrated into #94.

@smortex smortex closed this Aug 30, 2017
@kachick
Copy link
Copy Markdown
Contributor Author

kachick commented Aug 30, 2017

Thank you!

@kachick kachick deleted the improve-remove_warnings branch August 30, 2017 07:10
TheHolyRoger pushed a commit to whatagem/ipaddress that referenced this pull request Oct 18, 2021
Provide arithmetic operations on IPv4 and IPv6
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