search

laurilehmijoki / s3_website

Manage an S3 website: sync, deliver via CloudFront, benefit from advanced S3 website features.
Other
2.24k stars 186 forks source link

--headless on push doesn't seem to be supported... #114

Closed milesmatthias closed 10 years ago

milesmatthias commented 10 years ago 
± |master ✗| → s3_website push --site 'build' --headless
ERROR: s3_website push was called with arguments ["--headless"]
Usage: "s3_website push".

what am I doing wrong here?

milesmatthias commented 10 years ago

I'm using version 2.3.1 by the way.

I know this works on version 1.8.1.

milesmatthias commented 10 years ago

ah, I see you removed that feature... https://github.com/laurilehmijoki/s3_website/blob/84309deeb8bb17167764a70a1bcc4e7bee55832d/changelog.md#removed-features

Why?

laurilehmijoki commented 10 years ago

I removed the --headless option because one can use the _ignore_onserver and _exclude_fromupload instead of the manual "delete or keep the file" steps.

In addition, implementing the v2 was easier, because I didn't need to implement user prompting.

Furthermore, user prompting is a sign of incomplete automation.

What do you think about the reasoning above?

laurilehmijoki commented 10 years ago

Now that I think of it, we should add a graceful warning message for the users who run the v2 with the --headless option.

laurilehmijoki commented 10 years ago

Created issue #115.