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

Install troubleshooting on Ubuntu #5817

Merged
merged 5 commits into from Jan 27, 2017

Conversation

Projects
None yet
5 participants
@DirtyF
Member

DirtyF commented Jan 24, 2017

closes #5759

  • The link to the troubleshooting page is more obvious on installation page
  • Ubuntu troubleshooting is on troubleshooting page with other Linux specific instructions

/cc @BlueberryFoxtrot

BlueberryFoxtrot and others added some commits Jan 12, 2017

Update installation.md
1. Addition of *Running Jekyll on Ubuntu* section, to address Ubuntu stumbling block as per #5719.
2. Restructuring, and I hope I understood correctly when NodeJS/Python are/aren't required.
3. Gentler wording – it's probably not a good idea to tell punters who hit this page because they ran into trouble that installing Jekyll *is* easy and straight-forward; it *ought to be* straight-forward. (There's always the potential for pain and confusion if not all dependencies are in place.)

@DirtyF DirtyF requested a review from pathawks Jan 24, 2017

@DirtyF DirtyF requested a review from oe Jan 25, 2017

@DirtyF DirtyF added do-not-merge and removed do-not-merge labels Jan 25, 2017

Show outdated Hide outdated docs/_docs/installation.md
@@ -10,18 +10,29 @@ encountered and how we might make the process easier.
### Requirements
Installing Jekyll is easy and straight-forward, but there are a few
requirements you’ll need to make sure your system has before you start.
Installing Jekyll ought to be straight-forward if all requirements are met.

This comment has been minimized.

@oe

oe Jan 25, 2017

Member

can we replace ought to be with should be? it's a little less complicated

@oe

oe Jan 25, 2017

Member

can we replace ought to be with should be? it's a little less complicated

@oe

oe approved these changes Jan 25, 2017

@parkr

parkr approved these changes Jan 27, 2017

@DirtyF

This comment has been minimized.

Show comment
Hide comment
@DirtyF

DirtyF Jan 27, 2017

Member

@jekyllbot: merge +doc

Member

DirtyF commented Jan 27, 2017

@jekyllbot: merge +doc

@jekyllbot jekyllbot merged commit 1ff5902 into jekyll:master Jan 27, 2017

2 checks passed

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

jekyllbot added a commit that referenced this pull request Jan 27, 2017

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