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

Added a few clarifying comments #847

Merged
merged 1 commit into from
Feb 17, 2017
Merged

Conversation

ShaneCurcuru
Copy link
Contributor

YAML references are not always obvious, and ensuring people know to remove your Google IDs seemed a good idea to comment on. Thanks for minimal mistakes!

YAML references are not always obvious, and ensuring people know to remove your Google IDs seemed a good idea to comment on.  Thanks for minimal mistakes!
@mmistakes
Copy link
Owner

Thanks.

The "canonical" _config.yml is here... so I wasn't too worried about adding the comments to the documentation's site config.

But still a good addition. I know a few people like to use /docs as a starter... going so far as forgetting to remove my Google Adsense banners from their sites 😝

@mmistakes mmistakes merged commit 6cb699c into mmistakes:develop Feb 17, 2017
@ShaneCurcuru ShaneCurcuru deleted the patch-1 branch February 17, 2017 18:01
@ShaneCurcuru
Copy link
Contributor Author

Ha! I knew it. And d'oh for patching just the docs/ version.

kkunapuli pushed a commit to kkunapuli/kkunapuli.github.io that referenced this pull request May 30, 2019
makaroniame added a commit to makaroniame/makaroniame-old.github.io that referenced this pull request May 18, 2022
jchwenger pushed a commit to jchwenger/jchwenger.github.io that referenced this pull request May 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants