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

Comment typo. #7641

Merged
merged 1 commit into from Aug 29, 2014

Conversation

4 participants
@cirosantilli
Contributor

cirosantilli commented Aug 29, 2014

No description provided.

@TeatroIO

This comment has been minimized.

TeatroIO commented Aug 29, 2014

I've prepared a stage. Click to open.

@@ -4,7 +4,7 @@ class CompareService
def execute(current_user, source_project, source_branch, target_project, target_branch)
# Try to compare branches to get commits list and diffs
#
# Note: Use satellite only when need to compare between to repos
# Note: Use satellite only when need to compare between two repos

This comment has been minimized.

@houndci-bot

houndci-bot Aug 29, 2014

Avoid inline comments.

This comment has been minimized.

@Razer6

Razer6 Aug 29, 2014

Member

Shut up :P It seems we've to update the configuration.

This comment has been minimized.

@cirosantilli

cirosantilli Aug 29, 2014

Contributor

I was thinking the same here. Also you merged too fast! =) I was about to force push another typo on the same file but stopped to eat something... =( sorry for double MR #7642

This comment has been minimized.

@cirosantilli

cirosantilli Aug 29, 2014

Contributor

Should be fixed at: #7643

Razer6 added a commit that referenced this pull request Aug 29, 2014

@Razer6 Razer6 merged commit 8a7d10a into gitlabhq:master Aug 29, 2014

1 check was pending

continuous-integration/travis-ci The Travis CI build is in progress
Details

@Razer6 Razer6 referenced this pull request Aug 29, 2014

Merged

Upgrade RuboCop to 0.25.0 #363

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment