Add info about checking version + updating #5497

Merged
merged 4 commits into from Oct 19, 2016

Projects

None yet

6 participants

@kingscott
Contributor
kingscott commented Oct 19, 2016 edited

In regards to #5484. I added a section to /docs/installation/ to show how to check your version of Jekyll, find the latest version and update it.

Open to feedback.

@kingscott kingscott Add info about checking version + updating
8a4edc5
@kingscott kingscott Fix some typos
e6829b5
+
+```sh
+$ jekyll --version
+$ gem list jekyll
@pathawks
pathawks Oct 19, 2016 Member

If we're trying to encourage the use of Bundler everywhere, maybe we should change this to something like

bundle exec jekyll --version
bundle show jekyll

Maybe that requires a larger refactoring of this document than is in the scope of this PR

@kingscott
kingscott Oct 19, 2016 Contributor

That could be done! Was just trying to keep it scoped; open for change though 👍

@pathawks
pathawks Oct 19, 2016 Member

Probably best to take care of that in a separate PR 👍

site/_docs/installation.md
+$ gem list jekyll
+```
+
+You can also use <a href="https://rubygems.org/gems/jekyll">RubyGems</a> to find the current versioning of any gem. But you can also use the `gem` command line tool:
@pathawks
pathawks Oct 19, 2016 Member

Since this is Markdown, let's use Markdown style links:

[RubyGems](https://rubygems.org/gems/jekyll)
@kingscott kingscott Fix some typos
2be5d56
@parkr parkr was assigned by jekyllbot Oct 19, 2016
@parkr
parkr approved these changes Oct 19, 2016 View changes

💗

@pathawks
Member

Fixes #5484

@ashmaroli
Contributor

nice write-up @kingscott, may I ask you to include a line about gem outdated as well, for completion sakes? Thanks.

@kingscott kingscott Add comment about gem outdated
c7d0dda
@kingscott
Contributor
kingscott commented Oct 19, 2016 edited

Sure, how's that @ashmaroli?

@ashmaroli

Looks great! 👍

@kingscott
Contributor
kingscott commented Oct 19, 2016 edited

Awesome, thanks!

@kingscott
Contributor

Any idea about the errors, not sure why the build fails?

@ashmaroli
Contributor

Dont worry about the failing tests.. its a known bug.. totally unrelated to this PR.

@DirtyF
Member
DirtyF commented Oct 19, 2016

This only covers one way to update Jekyll, we can already publish this and complete with bundler method later.

@pathawks
Member

:shipit:

@parkr
Member
parkr commented Oct 19, 2016

@jekyllbot: merge +site

@jekyllbot jekyllbot merged commit 0484986 into jekyll:master Oct 19, 2016

1 of 2 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@kingscott
Contributor

Sweet, thanks guys!

@parkr
Member
parkr commented Oct 19, 2016

@kingscott Thank you! 😄

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