Add missing flag to disable the watcher #3820

Merged
merged 2 commits into from Aug 4, 2015

Conversation

Projects
None yet
4 participants
@chrisfinazzo
Contributor

chrisfinazzo commented Jun 25, 2015

No description provided.

@envygeeks

This comment has been minimized.

Show comment
Hide comment
@envygeeks

envygeeks Jun 25, 2015

Contributor

I personally like the way that CLI's (well some do some dont) where it's --[no-]watch but :shipit:

Contributor

envygeeks commented Jun 25, 2015

I personally like the way that CLI's (well some do some dont) where it's --[no-]watch but :shipit:

@chrisfinazzo

This comment has been minimized.

Show comment
Hide comment
@chrisfinazzo

chrisfinazzo Jun 25, 2015

Contributor

To each their own, I guess...

See #3607.

Contributor

chrisfinazzo commented Jun 25, 2015

To each their own, I guess...

See #3607.

@parkr

View changes

site/_docs/configuration.md
@@ -180,7 +180,7 @@ class="flag">flags</code> (specified on the command-line) that control them.
<p class="description">Enable auto-regeneration of the site when files are modified.</p>
</td>
<td class="align-center">
- <p><code class="flag">-w, --watch</code></p>
+ <p><code class="flag">-w, --watch, --no-watch</code></p>

This comment has been minimized.

@parkr

parkr Jun 25, 2015

Member

I agree with @envygeeks. Would love to see this be --[no-]watch or similar.

@parkr

parkr Jun 25, 2015

Member

I agree with @envygeeks. Would love to see this be --[no-]watch or similar.

This comment has been minimized.

@chrisfinazzo

chrisfinazzo Jun 25, 2015

Contributor

Sure, will update.

Out of curiosity, is there a short equivalent of the --no-watch flag? Seems odd to only have -w.

@chrisfinazzo

chrisfinazzo Jun 25, 2015

Contributor

Sure, will update.

Out of curiosity, is there a short equivalent of the --no-watch flag? Seems odd to only have -w.

This comment has been minimized.

@parkr

parkr Jun 25, 2015

Member

Out of curiosity, is there a short equivalent of the --no-watch flag? Seems odd to only have -w.

Nope.

@parkr

parkr Jun 25, 2015

Member

Out of curiosity, is there a short equivalent of the --no-watch flag? Seems odd to only have -w.

Nope.

parkr added a commit that referenced this pull request Aug 4, 2015

@parkr parkr merged commit 627af07 into jekyll:master Aug 4, 2015

1 check passed

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

parkr added a commit that referenced this pull request Aug 4, 2015

@chrisfinazzo chrisfinazzo deleted the chrisfinazzo:configuration-docs branch Nov 5, 2015

@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017

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