Skip to content
Permalink
Browse files

Improve Jekyll example (#464)

* Remove jekyll from the install step

If we're bundling anyway in the following step, installing jekyll is a
superfluous step.
,

* Only require installing ruby on production

now dev will complain about yum locally, otherwise

* Remove /public after build step completes

* Update site build directory

* Update jekyll/package.json

Co-Authored-By: Steven <steven@ceriously.com>

* Apply suggestions from code review
  • Loading branch information...
sarupbanskota authored and msweeneydev committed Aug 8, 2019
1 parent 9b0399d commit 862c9a4c0221b7409f1baeeaf5f9293a1272bc2f
Showing with 6 additions and 2 deletions.
  1. +1 −0 jekyll/.gitignore
  2. +2 −0 jekyll/_config.yml
  3. +3 −2 jekyll/package.json
@@ -1,3 +1,4 @@
_site
public
.sass-cache
.jekyll-metadata
@@ -23,6 +23,8 @@ baseurl: "" # the subpath of your site, e.g. /blog
url: "" # the base hostname & protocol for your site, e.g. http://example.com
twitter_username: jekyllrb
github_username: jekyll
destination: public
permalink: pretty

# Build settings
markdown: kramdown
@@ -1,6 +1,7 @@
{
"private": true,
"scripts": {
"install": "yum install ruby23-devel.x86_64 && gem install jekyll bundler --no-ri --no-rdoc && bundle install",
"build": "jekyll build && mv _site public"
"install": "yum install ruby23-devel.x86_64 && gem install bundler --no-ri --no-rdoc && bundle install",
"build": "jekyll build"
}
}

1 comment on commit 862c9a4

@now

This comment has been minimized.

Copy link

commented on 862c9a4 Aug 8, 2019

Please sign in to comment.
You can’t perform that action at this time.