Need to update --full-rebuild tag to something like --incremental or --incr.

Is there any documentation on this feature? Perhaps it can say it's off by default.

On it!

There's barely any documentation, though it prints out whether it's enabled when building:

Configuration file: /Users/alfred/git/jekyll/test/_config.yml
            Source: /Users/alfred/git/jekyll/test
       Destination: /Users/alfred/git/jekyll/test/_site
 Incremental build: disabled
      Generating...

Incremental build: disabled

Maybe we should change this to say disabled. Enable with --incremental? or whatever flag you decide on

/cc envygeeks/jekyll-docker#51 envygeeks/jekyll-assets#191

I prefer a switch/command/trigger to enable regeneration in general regardless of current state.

Building, Serving, Regenerating... all specific and chosen capabilities.

I would expect jekyll build to build once and return successful.

I would expect to jekyll build ---watch or jekyll build -w to enable/turn on regeneration.

$.02 💸

@jaybe-jekyll that very well could be the case when it's enabled by default but with it being disabled by default as an experimental feature now, that won't work.

@jaybe-jekyll It's still very experimental, but that's a fantastic idea for the future! :) Especially because -w is on by default for jekyll serve.

Agreed! Commits coming as soon as the tests pass. It'll be incremental in configuration and --incremental or -ir as command-line flags.

I need to correct myself; apologies --

I prefer a switch/command/trigger required to enable regeneration,

Building, Serving, Regenerating... all specific and chosen capabilities.

I would expect jekyll build to build once and end, returning as successful.

I would expect to jekyll build --regnerateto enable/turn on regeneration, with or without --watch, as --regeneration would presume --watch.

Pfew.

$.02 💸

I think -ir might get confusing. Maybe --ir or use just one character for a single dash.

@parkr: Would you prefer -i or -r for the shorthand flag?

@jaybe-jekyll Not sure about the "flag presumes --watch" part, since incremental regen would offer a performance improvement even without watching (so users might want to enable it without enabling watch)

Another opinion; I think the term, "incremental", in "incremental regeneration" (i.e. --ir) applies more to a technical audience and isn't necessarily necessary or appropriate for general use(rs).

Regenerate.

Agreed; single character enablement; perhaps -R (for, REGENERATION POWERS ACTIVATED). And, for the verbose and visually-minded, --regenerate and or --regeneration.

@parkr: Would you prefer -i or -r for the shorthand flag?

Historically, I have used -i for "case-insensitive", and -r for "recursive." I would go with the first character of the double-dashed one or an alternative if it isn't there.

@alfredxing I don't see how generation could occur within a system without --watch; does it not imply watching for changes (--watch) then acting on those changes (--regenerate)?

@parkr @alfredxing: @jaybe-jekyll brings up a fine point, why not call the flag --cache? That's literally what it's doing, but we call it incremental regeneration as a technical term, when Occam is standing over there pointing to the term: caching.

Does the sass compiler have a flag like this? Our Jekyllers tend to be front end devs so maybe Sass's CLI is known.

@jaybe-jekyll Yeah, it's more about caching. --watch says "hey, regenerate every time you detect a changed file!" and --regenerate or --increment or --cache is "hey, when you do regenerate, only regenerate the changed files!". So they're different functions entirely. Eventually, --cache or whatever will be enabled by default, no matter if --watch is used.

Sass by default caches so they have a --force flag which forces it to update the cache even if it's cached.

@alfredxing just waiting on this PR. once it's merged, i'll release an RC and then do any final bug fixes before I release around 8pm PST tonight.

Sure! Right now I have it as -I and --incremental. If we don't think -I is appropriate, I can remove it!

-I is what I instantly thought of when you said you wanted --incremental so I'm 👍

Looks great! Thanks so much, @alfredxing. I'll merge once it's 💚 .

👍

Good luck (& congrats!) on the launch tonight!

Good luck (& congrats!) on the launch tonight!

@alfredxing Congrats to you, too! It's been a huge team effort. Thanks for all your time and effort thus far – it's meant a lot!