Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix post date #2394

Merged
merged 1 commit into from
May 11, 2014
Merged

Fix post date #2394

merged 1 commit into from
May 11, 2014

Conversation

doktorbro
Copy link
Member

Instead of changing the reference to the post like in #2367, I prefer to fix the date of the post itself. Note that the link http://jekyllrb.com/news/2013/05/05/jekyll-1-0-0-released/ doesn’t change.

And I changed the time zone to Los Angeles like @parkr suggested in #2367 (comment).

@parkr
Copy link
Member

parkr commented May 11, 2014

I like the change to _config.yml, but not to the post. Having the exact time is important to me and keeps the time consistent across all timezones, such that it occurred at the same moment in time (not shifted by timezone).

@doktorbro
Copy link
Member Author

@parkr Post changes are reverted. The docs are in Los Angeles now.

@parkr
Copy link
Member

parkr commented May 11, 2014

Cool, thanks. Can you please squish those down into 1 commit?

@doktorbro
Copy link
Member Author

Can you please squish those down into 1 commit?

Done.

parkr added a commit that referenced this pull request May 11, 2014
@parkr parkr merged commit 8068702 into jekyll:master May 11, 2014
parkr added a commit that referenced this pull request May 11, 2014
@doktorbro doktorbro deleted the fix-dates branch May 12, 2014 04:53
@parkr parkr mentioned this pull request May 12, 2014
@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants