Make noTimes command flag work #1657

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
@tatsushid
Contributor

tatsushid commented Dec 1, 2015

Hugo command provides "noTimes" flag but it doesn't affect synced files
timestamp regardless of whether the flag is specified or not.

This fixes it by adding a flag check and overwriting configuration if it
is specified.

Fix #1583

Make noTimes command flag work
Hugo command provides "noTimes" flag but it doesn't affect synced files
timestamp regardless of whether the flag is specified or not.

This fixes it by adding a flag check and overwriting configuration if it
is specified.

Fix #1583
@anthonyfok

This comment has been minimized.

Show comment
Hide comment
@anthonyfok

anthonyfok Dec 1, 2015

Member

Thanks again @tatsushid for a great fix!

Merged as e989468 (master).

Member

anthonyfok commented Dec 1, 2015

Thanks again @tatsushid for a great fix!

Merged as e989468 (master).

@anthonyfok anthonyfok closed this Dec 1, 2015

anthonyfok referenced this pull request Dec 3, 2015

Change most global flags into local ones
This is to ensure that only the relevant command-line flags
for a certain Hugo subcommand is shown to the end user,
reducing clutter and improving user experience.

Fixes #1624 - CLI UX: Flags shouldn't be global
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment