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.

Sign up for GitHub

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

Jump to bottom

Added missing dependencies #133

Closed
wants to merge 1 commit into from
Closed

Added missing dependencies #133

wants to merge 1 commit into from

Conversation

KrauseFx
Copy link
Member

image

@coveralls
Copy link

Coverage Status

Coverage remained the same at 61.01% when pulling c101ee7 on KrauseFx:patch-1 into e14419a on neonichu:master.

@KrauseFx
Copy link
Member Author

Coveralls says it's fine, and tests pass. Ready to be merged and shipped

@KrauseFx
Copy link
Member Author

Why is this not merged yet? This is a high priority fix

@KrauseFx
Copy link
Member Author

Closing due to inactivity.

@KrauseFx KrauseFx closed this May 23, 2016
@KrauseFx KrauseFx deleted the patch-1 branch May 23, 2016 18:48
@neonichu
Copy link
Collaborator

Was just about to merge, plz resubmit, need Rails 🚋

@KrauseFx
Copy link
Member Author

Ha, we have rocket technology 🚀

@neonichu
Copy link
Collaborator

Best PR ever, so sad that you closed it

@KrauseFx
Copy link
Member Author

😢

@neonichu
Copy link
Collaborator

@KrauseFx plz update

@KrauseFx
Copy link
Member Author

⚡

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@KrauseFx @coveralls @neonichu