-
Notifications
You must be signed in to change notification settings - Fork 206
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
update documentation of how to run the code #554
base: master
Are you sure you want to change the base?
update documentation of how to run the code #554
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good, but a few things to fix, thanks.
@@ -82,11 +82,13 @@ cd www | |||
### Installation | |||
Install Ruby and the Gem package manager (pre-installed in Mac OS X) and install `jekyll` and its related packages available in the Github hosted version by running, from this directory: | |||
|
|||
bundle install; | |||
bundle install |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
### Running | ||
Jekyll can run a local web server that rebuilds each time you save a page while editing (Execute the below command from root directory): | ||
|
||
bundle exec jekyll serve --incremental; | ||
`jekyll serve`: rebuilds the entire website from scratch each time. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing bundle exec
.
Narrative is up against the command. Please break the line, as is the style now in use.
bundle exec jekyll serve --incremental; | ||
`jekyll serve`: rebuilds the entire website from scratch each time. | ||
|
||
`bundle exec jekyll serve --incremental`:Only regenerates changed files since the last build, making development faster and more efficient but does not render posts (i.e. FAQ, blog, news, and events). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Narrative is up against the command. Please break the line, as is the style now in use.
Your commit message could be simpler; e.g. Add Jekyll incremental vs full serve running docs.
@pikurasa Can you please review it.