fix(blog): post front matter fails to parse if date has a time component. #17

Open
katie wants to merge 1 commit from katie/flora:yaml_time_parsing into live
Collaborator
No description provided.
fix(blog): post front matter fails to parse if date has a time component.
All checks were successful
ci/woodpecker/pr/woodpecker/1 Pipeline was successful
ci/woodpecker/pr/woodpecker/2 Pipeline was successful
75cbdd7e88
Author
Collaborator

Actually, this started blowing up my builds with undefined method 'to_time' for an instance of String (NoMethodError) in Post.date, I haven't figured out why.

Actually, this started blowing up my builds with `undefined method 'to_time' for an instance of String (NoMethodError)` in `Post.date`, I haven't figured out why.
Author
Collaborator

The parser requires ISO 8601 timestamps, so if you specify a time, it needs to include seconds. Which I guess is fine, there's just no good way to document that (other than maybe handling it specifically within Post.date)

The parser requires ISO 8601 timestamps, so if you specify a time, it needs to include seconds. Which I guess is fine, there's just no good way to document that (other than maybe handling it specifically within `Post.date`)
katie force-pushed yaml_time_parsing from 75cbdd7e88
All checks were successful
ci/woodpecker/pr/woodpecker/1 Pipeline was successful
ci/woodpecker/pr/woodpecker/2 Pipeline was successful
to 71f04167fd
Some checks are pending
ci/woodpecker/pr/woodpecker/1 Pipeline is pending approval
ci/woodpecker/pr/woodpecker/2 Pipeline is pending approval
2026-07-24 10:17:46 -04:00
Compare
Some checks are pending
ci/woodpecker/pr/woodpecker/1 Pipeline is pending approval
ci/woodpecker/pr/woodpecker/2 Pipeline is pending approval
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u yaml_time_parsing:katie-yaml_time_parsing
git switch katie-yaml_time_parsing

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch live
git merge --no-ff katie-yaml_time_parsing
git switch katie-yaml_time_parsing
git rebase live
git switch live
git merge --ff-only katie-yaml_time_parsing
git switch katie-yaml_time_parsing
git rebase live
git switch live
git merge --no-ff katie-yaml_time_parsing
git switch live
git merge --squash katie-yaml_time_parsing
git switch live
git merge --ff-only katie-yaml_time_parsing
git switch live
git merge katie-yaml_time_parsing
git push origin live
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
nick/flora!17
No description provided.