Fix — entrées portfolio LabML invisibles : date de front matter dans le futur - #23
Merged
Conversation
… future Jekyll skips future-dated documents: 07:00 -0400 was still hours away at build time, so both LabML pages 404'd while their images deployed. Moved the date to 01:00 -0400 (already past). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UKw6oNC8iZ9Kn7q6x4qom4
There was a problem hiding this comment.
Pull request overview
This PR fixes two portfolio pages that were not being generated by Jekyll because their front matter date was set in the future at build time, causing the resulting /portfolio/.../ routes to return 404 even though static assets were deployed.
Changes:
- Adjusts the
datein the EN LabML portfolio entry to an earlier (non-future) timestamp. - Adjusts the
datein the FR LabML portfolio entry to an earlier (non-future) timestamp.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
_portfolio/labml-ml-in-your-browser.md |
Moves front matter date earlier so the document is no longer excluded as “future” during builds. |
_portfolio/labml-le-ml-dans-votre-navigateur.md |
Same fix as EN: ensures the FR document is generated by avoiding a future date. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Quoi
Après le merge de #22, les deux pages LabML répondaient 404 alors que leurs images étaient bien déployées. Cause : la date de front matter (
2026-08-21 07:00:00 -0400, soit 11:00 UTC) était encore dans le futur au moment du build — Jekyll exclut les documents futurs, donc les pages n'étaient pas générées, mais les assets statiques, eux, partaient.Correctif d'une ligne par fichier : date reculée à
01:00:00 -0400(déjà passée). Au prochain build Pages,/portfolio/labml-ml-in-your-browser/et/portfolio/labml-le-ml-dans-votre-navigateur/seront servies.Generated by Claude Code