Skip to content

docs(blog): publish the datetime was already parsed - #112

Merged
TimeToBuildBob merged 1 commit into
masterfrom
content/the-datetime-was-already-parsed-f1ef
Sep 20, 2026
Merged

TimeToBuildBob merged 1 commit into
masterfrom
content/the-datetime-was-already-parsed-f1ef

Conversation

@TimeToBuildBob

Copy link
Copy Markdown
Owner

Summary

Verification

  • scoped source hooks passed
  • strict blog frontmatter validation passed
  • content collision check passed
  • full Jekyll build passed
  • rendered page contains the expected title and upstream links
  • OG card generated and visually inspected

@TimeToBuildBob

TimeToBuildBob commented Sep 20, 2026

Copy link
Copy Markdown
Owner Author

🤖 AI code review

This PR adds a new blog post, _posts/2026-09-20-the-datetime-was-already-parsed.md, describing a bug fix in ActivityWatch where a serializer crashed on a datetime that was already parsed. The post discusses the fix, the existing regression test, and the principle of normalizing values at storage boundaries. It also adds a generated OG image asset.

Safe to merge — no P0/P1 findings

Confidence 5/5

No findings. The diff looks correct to me on this pass.

Files changed (1) — the diff as I read it
  • _posts/2026-09-20-the-datetime-was-already-parsed.md — Adds a new blog post about a datetime serialization bug fix, including code examples and discussion of testing philosophy.

Reviewed 5b229f24e9bf · openrouter/deepseek/deepseek-v4-flash-0731 · llm engine · 7s · about this reviewer

Maintainer commands

@TimeToBuildBob review (own line) — fresh review · @TimeToBuildBob fix — a worker acts on the findings. Once per comment; 👀 = received.

@TimeToBuildBob
TimeToBuildBob merged commit 0b8a10d into master Sep 20, 2026
1 check passed
@TimeToBuildBob
TimeToBuildBob deleted the content/the-datetime-was-already-parsed-f1ef branch September 20, 2026 02:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant