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

Docs: GitHub Pages instructions #6384

Merged
merged 1 commit into from Sep 25, 2017

Conversation

Projects
None yet
5 participants
@DirtyF
Member

DirtyF commented Sep 25, 2017

  • Clarify pages-gem intent (fix #6187)
  • Update Gemfile example,
  • Use URL filters for links to assets.
  • Move up link to GitHub Pages Help and Support.

@DirtyF DirtyF requested a review from benbalter Sep 25, 2017

Show outdated Hide outdated docs/_docs/github-pages.md Outdated
Show outdated Hide outdated docs/_docs/github-pages.md Outdated
Show outdated Hide outdated docs/_docs/github-pages.md Outdated
Show outdated Hide outdated docs/_docs/github-pages.md Outdated
Show outdated Hide outdated docs/_docs/github-pages.md Outdated
Show outdated Hide outdated docs/_docs/github-pages.md Outdated
Show outdated Hide outdated docs/_docs/github-pages.md Outdated
will resolve properly.
## Deploying Jekyll to GitHub Pages
GitHub Pages work by looking at certain branches of repositories on GitHub.
There are two basic types available: user/organization pages and project pages.
There are two basic types available: [user/organization and project pages](https://help.github.com/articles/user-organization-and-project-pages/).
The way to deploy these two types of sites are nearly identical, except for a

This comment has been minimized.

@ashmaroli

ashmaroli Sep 25, 2017

Member

split line at around 80 chars.. ?

@ashmaroli

ashmaroli Sep 25, 2017

Member

split line at around 80 chars.. ?

Docs: GitHub Pages instructions
- Clarify `pages-gem` intent
- Update Gemfile example
- Use URL filters for links to assets
@DirtyF

This comment has been minimized.

Show comment
Hide comment
@DirtyF

DirtyF Sep 25, 2017

Member

Thanks for the feedback, incorporated your suggestions.

Member

DirtyF commented Sep 25, 2017

Thanks for the feedback, incorporated your suggestions.

@oe

oe approved these changes Sep 25, 2017

@benbalter

🎉

@DirtyF

This comment has been minimized.

Show comment
Hide comment
@DirtyF

DirtyF Sep 25, 2017

Member

@jekyllbot: merge +docs

Member

DirtyF commented Sep 25, 2017

@jekyllbot: merge +docs

@jekyllbot jekyllbot merged commit e4b456a into master Sep 25, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jekyllbot jekyllbot deleted the docs-pages-gem branch Sep 25, 2017

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