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

Rubocop fixes - I'd buy that for a dollar! #1258

Merged
merged 10 commits into from
Nov 7, 2016
Merged

Conversation

jaredmoody
Copy link
Contributor

@jaredmoody jaredmoody commented Nov 5, 2016

A collection of autocorrected violation fixes for #945

@jasonrclark
Copy link
Member

This is looking good to me. ✨

I'm just about ready to cut pre6 off #1252 so I'd like to hold this until that lands. Since a lot of these automated changes are in the samples, if they do have subtle breakage our tests wouldn't necessarily catch them.

Thanks again for all the tidying @jaredmoody 🛁

@jaredmoody
Copy link
Contributor Author

👍 I've got more PRs ready as soon as pre6 is done

@PragTob
Copy link
Member

PragTob commented Nov 7, 2016

Well then, let's get this in :)

@PragTob PragTob merged commit 203f637 into shoes:master Nov 7, 2016
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