Skip to content
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

Minor typographical fixes #18

Merged
merged 1 commit into from
May 22, 2013
Merged

Minor typographical fixes #18

merged 1 commit into from
May 22, 2013

Conversation

sethfri
Copy link
Contributor

@sethfri sethfri commented May 22, 2013

On the CocoaPods website's main specification page, single quotes are used for the source URL and the tag. This pull request aims to make that consistent in the source documentation.

@coveralls
Copy link

Coverage Status

Coverage increased (+0%) when pulling 371cba6 on sethfri:master into 34b3cd1 on CocoaPods:master.

# spec.source = { :git => "git://github.com/AFNetworking/AFNetworking.git",
# :tag => "v#{spec.version}" }
# spec.source = { :git => 'git://github.com/AFNetworking/AFNetworking.git',
# :tag => 'v#{spec.version}' }
Copy link
Member

Choose a reason for hiding this comment

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

The interpolation doesn't work with the single quotes.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should v#{spec.version} be in double quotes then?

Copy link
Member

Choose a reason for hiding this comment

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

Yes, otherwise people less familiar with Ruby might get confused by this example.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

@coveralls
Copy link

Coverage Status

Coverage increased (+0%) when pulling 66d6418 on sethfri:master into 34b3cd1 on CocoaPods:master.

@fabiopelosin
Copy link
Member

👍 Thanks!

fabiopelosin added a commit that referenced this pull request May 22, 2013
Minor typographical fixes
@fabiopelosin fabiopelosin merged commit a48928c into CocoaPods:master May 22, 2013
Ashton-W pushed a commit to Ashton-W/Core that referenced this pull request Nov 2, 2015
amorde added a commit that referenced this pull request Jan 24, 2019
…tation

Closes #18

The documentation for this attribute seems pretty substantial, this adds the last item
mentioned on that issue.
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.

None yet

3 participants