Skip to content

Conversation

twiest
Copy link

@twiest twiest commented May 25, 2014

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling d39b0d1 on twiest:fix_bug_98 into 4a31d40 on ManageIQ:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 5b15a1c on twiest:fix_bug_98 into 4a31d40 on ManageIQ:master.

@cfme-bot
Copy link
Member

Checked commits https://github.com/twiest/linux_admin/commit/d39b0d171edc5f42e70a7fed1ea595f4ff7617f1 .. https://github.com/twiest/linux_admin/commit/5b15a1c2018e251b76c60c88f965e968302f92d9 with rubocop 0.21.0
2 files checked, 0 offenses detected
Everything looks good. ⭐

@Fryguy
Copy link
Member

Fryguy commented Jun 3, 2014

Can you squash your commits and change the title to describe the issue being fixed as opposed to just a reference to the bug? In addition, put a reference to the bug like Fixes #98 in the body of the commit.

Thanks for your contribution!

Fryguy pushed a commit to Fryguy/linux_admin that referenced this pull request Jul 9, 2014
@Fryguy Fryguy closed this in 3adf11e Jul 10, 2014
jrafanie added a commit that referenced this pull request Jul 10, 2014
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.

4 participants