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

Adding Rubocop. Moving to use gemspec. Add rubocop to travis run. #30

Merged
merged 3 commits into from
Jun 11, 2015

Conversation

housepage
Copy link
Contributor

Needed to do a rebase of @michaelachrisco work in #26 for it to mergeable. Also added rubocop run to travis build.

  • Fix Gem Gemfile to bundle with soda.gemspec
  • Add appropriate packages to Gemspec
  • Rubocop first pass
    • 5 offenses left
  • Add rubocop run to travis (but prevent it from failing the build)

@housepage housepage mentioned this pull request Jun 11, 2015
@michaelachrisco
Copy link
Contributor

Cool thanks. I was in the middle of a meeting so this saves me a bit of work 😄

@housepage housepage force-pushed the michaelachrisco/rubocop_best_practice branch from adc7ee7 to 448d0a8 Compare June 11, 2015 23:22
housepage added a commit that referenced this pull request Jun 11, 2015
…tice

Adding Rubocop. Moving to use gemspec. Add rubocop to travis run.
@housepage housepage merged commit 66825cb into master Jun 11, 2015
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