voxpelli / node-format-microformat

Formats a Microformat JSON representation into eg. a Jekyll post
MIT License
13 stars 9 forks source link

Adjust default publish time to be a second or so older than actual time #4

Closed voxpelli closed 7 years ago

voxpelli commented 8 years ago

If Jekyll is not configured with future: true then posts from the future will not be published.

To avoid a mismatch in server times to cause this it could be wise to have the default publish time be a second or so older so that it for sure will be considered a non-future post on the Jekyll build server as well.

I think I have encountered such a scenario on my GitHub Pages blog, where API-created events often weren't published and where the probable cause was that GitHub Pages considered them to be from the future and thus not yet ready to be published.

Thoughts on this change @miklb?

miklb commented 8 years ago

I don't see a problem with a few seconds