Adds a clean option to the generate command.

[davereid]

I thought it would be good if there was an option to clear the output directory out first prior to generation.

[beryllium]

Good idea!

👍

Does it need any safety checking on the docroot before it goes and deletes stuff, or can we be reasonably sure that it's not going to wipe out / or ~/? :)

[davereid]

I also debated just having this as a separate command too.

Hrm, I'm not sure about sanity checking. I would assume that the sculpin.output_dir is checked elsewhere by this point, but I'm not sure.

[beryllium]

I think it makes more sense as a parameter than a separate command. You can always just delete output_* by hand, but you'll often forget to do that on generate - if you've renamed any posts or pages or whatnot, this parameter can save you from the resulting cruft.

Also, it will help simplify people's publishing scripts.

[davedevelopment]

👍 for a simple check on the output_dir, even if it's printing the dir to screen with a Y/N input prompt just to be sure

[davereid]

I'd need a way to bypass the input prompt for my deployment workflow, so I'm a little at a loss for what checking I should put in.

[davereid]

I guess maybe adding a new --force parameter that would bypass the input prompt?

[simensen]

👍 for prompt + a way to bypass. I think there is already a non-interactive flag built into symfony console? Maybe we can use that? I think that is just supposed to accept defaults if defaults are allowed, right?

[davereid]

Ah, I see it now, --no-interaction

[davereid]

Ok, will work on the prompt.

[davereid]

Ok I think this is ready again for review, with the added question for confirmation if the --no-interaction flag is not used. I tested this myself with a manually built sculpin.phar.

[davedevelopment]

👍 from me, works all good.

[davereid]

Is there anything I can do to help this along? Anything missing? Does it need tests?

[davedevelopment]

@davereid nope, people have had enough time to object now, merged.