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

Check social_media_url changed from default value #85

Merged
merged 2 commits into from
Apr 1, 2014
Merged

Check social_media_url changed from default value #85

merged 2 commits into from
Apr 1, 2014

Conversation

dlackty
Copy link
Contributor

@dlackty dlackty commented Apr 1, 2014

This is a trivial work by just duplicating the logic and spec from homepage check.

This closes #67. :octocat: :octocat: :octocat:

@@ -7,7 +16,7 @@
* The specification now strips the indentation of the `prefix_header` and
`prepare_command` to aide their declaration as a here document (similarly to
what it already does with the description).
[Fabio Pelosin][irrationalfab]
[Fabio Pelosin](https://github.com/irrationalfab)
Copy link
Member

Choose a reason for hiding this comment

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

This was fine because the link is declared at bottom of the file 😄. However don't worry it works this way as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops. Didn't notice that. Should I change back?
Just learned a new feature of Markdown. Thanks man! 😉

Copy link
Member

Choose a reason for hiding this comment

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

No need to change it back, it still works 🏄

@fabiopelosin
Copy link
Member

This looks good to me, I'm just waiting for Travis which is being a bit slow today https://travis-ci.org/CocoaPods/Core/pull_requests.

Thanks for the contribution!

@dlackty
Copy link
Contributor Author

dlackty commented Apr 1, 2014

You're welcome. However, I forced pushed the commit and removed that change.
Feel so glad to contribute to this project! 😄

@fabiopelosin
Copy link
Member

Nice!

Nice

Merging!

fabiopelosin added a commit that referenced this pull request Apr 1, 2014
Check social_media_url changed from default value
@fabiopelosin fabiopelosin merged commit 17bc1d3 into CocoaPods:master Apr 1, 2014
Ashton-W pushed a commit to Ashton-W/Core that referenced this pull request Nov 2, 2015
Check social_media_url changed from default value
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.

[Linter] check wether social_media_url is set to the example value
2 participants