Skip to content
This repository has been archived by the owner on Mar 16, 2021. It is now read-only.

Fixes #9708 - Fixing rubocop config so it can be used with houndci #97

Merged
merged 1 commit into from
Mar 13, 2015

Conversation

daviddavis
Copy link

Fixes the problem in #98

@parthaa
Copy link
Contributor

parthaa commented Mar 13, 2015

apj

daviddavis added a commit that referenced this pull request Mar 13, 2015
Fixes #9708 - Fixing rubocop config so it can be used with houndci
@daviddavis daviddavis merged commit 129c20b into theforeman:master Mar 13, 2015
@dLobatog
Copy link
Member

@daviddavis Please cherry-pick when merging instead of making a merge commit, it can easily make branching for a release quite complicated, and also testing compatibility with previous core versions more difficult. I'll revert this and merge it again, let me know if you have questions or any concerns with the process.

@daviddavis
Copy link
Author

@elobato I know. I just forgot. If you look through our closed PRs, I've been cherrypicking them (aside from this one). In Katello, we merge PRs so chances are I'll probably forget again. Not that I disagree with the process, just it's hard to keep everything straight when working on two different projects with different processes.

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

Successfully merging this pull request may close these issues.

4 participants