Skip to content

update Rubocop and Travis config #110

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

Merged
merged 1 commit into from
Oct 22, 2017
Merged

update Rubocop and Travis config #110

merged 1 commit into from
Oct 22, 2017

Conversation

jekyllbot
Copy link
Contributor

PR automatically created for @DirtyF.

update Rubocop and Travis config


- ran rubocop -a
- add remaining offenses to .rubocop.yml
- add script/fmt
- run script/fmt in script/cibuild
- update Travis rubies config

- ran rubocop -a
- add remaining offenses to .rubocop.yml
- add script/fmt
- run script/fmt in script/cibuild
- update Travis rubies config
@DirtyF DirtyF changed the title update Rubocop and Travis config - ran rubocop -a - add remaining offenses to .rubocop.yml - add script/fmt - run script/fmt in script/cibuild - update Travis rubies config update Rubocop and Travis config Oct 22, 2017
@DirtyF DirtyF requested a review from pathawks October 22, 2017 09:02
@@ -1,9 +1,15 @@
inherit_gem:
jekyll: .rubocop.yml

Lint/ShadowingOuterLocalVariable:
Exclude:
- lib/jekyll-archives.rb
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I probably want to fix this, but can do that in a separate PR

@DirtyF
Copy link
Member

DirtyF commented Oct 22, 2017

@jekyllbot: merge +dev

@jekyllbot jekyllbot merged commit a164d0e into master Oct 22, 2017
@jekyllbot jekyllbot deleted the pull/rubocop-travis branch October 22, 2017 13:36
jekyllbot added a commit that referenced this pull request Oct 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants