Skip to content
This repository has been archived by the owner on Dec 4, 2023. It is now read-only.

Removed Ruby warnings #253

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Removed Ruby warnings #253

wants to merge 3 commits into from

Conversation

tbuehlmann
Copy link

Removed some Ruby warnings. Though I couldn't test them, so beware.

Also, I found two cases where I'm unsure about:

@cowboyd
Copy link
Collaborator

cowboyd commented Jan 3, 2014

This is causing testing failures. You can run the tests by running "rake spec"

@tbuehlmann
Copy link
Author

Wanna give it another try? Should be semantically the same as the existing code now, too.

@mattconnolly
Copy link

Can one of the admins verify this patch?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants