Skip to content
This repository was archived by the owner on May 12, 2018. It is now read-only.

Typo herlper -> helper. #43

Merged
merged 1 commit into from
Sep 15, 2014
Merged

Conversation

cirosantilli
Copy link
Contributor

Maybe there is a reason for this, in that case I'm very curious to know =)

@@ -1,7 +1,7 @@
# Gitlab::Git::Repository is a wrapper around native Grit::Repository object
# We dont want to use grit objects inside app/
# It helps us easily migrate to rugged in future
require_relative 'encoding_herlper'
require_relative 'encoding_helper'

Choose a reason for hiding this comment

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

Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) when pulling 69a840a on cirosantilli:typo-herlper into 4883a55 on gitlabhq:master.

@@ -16,7 +16,7 @@

# Gitlab::Git
require_relative "gitlab_git/popen"
require_relative "gitlab_git/encoding_herlper"
require_relative 'gitlab_git/encoding_helper'

Choose a reason for hiding this comment

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

Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) when pulling 12a764d on cirosantilli:typo-herlper into 4883a55 on gitlabhq:master.

dzaporozhets added a commit that referenced this pull request Sep 15, 2014
@dzaporozhets dzaporozhets merged commit 732baeb into gitlabhq:master Sep 15, 2014
@dzaporozhets
Copy link
Contributor

nice typo :)

@cirosantilli cirosantilli deleted the typo-herlper branch September 15, 2014 15:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants