Skip to content

[News] Fix darkthumbnail image path in Docker Kanvas post - #8345

Merged
miacycle merged 2 commits into
layer5io:masterfrom
ayushbuilds-code:fix/8344-docker-kanvas-news-darkthumbnail
Nov 9, 2026
Merged

miacycle merged 2 commits into
layer5io:masterfrom
ayushbuilds-code:fix/8344-docker-kanvas-news-darkthumbnail

Conversation

@ayushbuilds-code

@ayushbuilds-code ayushbuilds-code commented Nov 8, 2026 •

Copy link
Copy Markdown
Contributor

Description

This PR fixes #8344

In src/collections/news/2026/2026-01-21-docker-kanvas-kubernetes-dominance/index.mdx, darkthumbnail was pointing to ./docker-kanvas-infoq.png, which does not exist in the directory (the asset was converted to .webp). This updates darkthumbnail to point to ./docker-kanvas-infoq.webp matching thumbnail.

Notes for Reviewers

  • Only changes the file extension in darkthumbnail from .png to .webp in src/collections/news/2026/2026-01-21-docker-kanvas-kubernetes-dominance/index.mdx.

Signed commits

  • Yes, I signed my commits.

Summary by CodeRabbit

  • Content
    • Updated the article’s thumbnail image.

@github-actions github-actions Bot added the area/news A noteworthy article, event, happening label Nov 8, 2026
@coderabbitai

coderabbitai Bot commented Nov 8, 2026 •

Copy link
Copy Markdown
Contributor

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The Docker Kanvas news post now points its darkthumbnail metadata to the WebP image instead of the PNG image.

Changes: News thumbnail

Layer / File(s) Summary
Correct the dark thumbnail path
src/collections/news/2026/2026-01-21-docker-kanvas-kubernetes-dominance/index.mdx
The darkthumbnail path changes from docker-kanvas-infoq.png to docker-kanvas-infoq.webp.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Bug fix · Severity of issue fixed: Low

Suggested labels: area/news

Merge Risk: Minimal · previous review a97ac

The thumbnail path correction has no identified merge-blocking risk and is ready to merge after normal checks.

Pre-merge checks: Passed 5

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title '[News] Fix darkthumbnail image path in Docker Kanvas post' matches the changeset. The only change updates the darkthumbnail metadata in the Docker Kanvas news post from docker-kanvas-infoq.…
Linked Issues check Passed Issue #8344 requires that darkthumbnail in src/collections/news/2026/2026-01-21-docker-kanvas-kubernetes-dominance/index.mdx point to ./docker-kanvas-infoq.webp. The reviewed head changes line 7…
Out of Scope Changes check Passed The whole-PR diff against the merge base modifies one line in one file: the darkthumbnail frontmatter value in the linked post. This change is the exact change requested by #8344. No other files or …
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Full details: Title check

Explanation

The title '[News] Fix darkthumbnail image path in Docker Kanvas post' matches the changeset. The only change updates the darkthumbnail metadata in the Docker Kanvas news post from docker-kanvas-infoq.png to docker-kanvas-infoq.webp, which the linked issue #8344 requests. The title names the file property, the problem area, and the post, so a teammate scanning history can identify the change.

Full details: Linked Issues check

Explanation

Issue #8344 requires that darkthumbnail in src/collections/news/2026/2026-01-21-docker-kanvas-kubernetes-dominance/index.mdx point to ./docker-kanvas-infoq.webp. The reviewed head changes line 7 from ./docker-kanvas-infoq.png to ./docker-kanvas-infoq.webp, which matches the requirement. The file docker-kanvas-infoq.webp exists in the post directory at the reviewed head, so the reference resolves. The issue does not request automated tests or other coding work. The whole-PR diff against the merge base contains only this one-line change.

Full details: Out of Scope Changes check

Explanation

The whole-PR diff against the merge base modifies one line in one file: the darkthumbnail frontmatter value in the linked post. This change is the exact change requested by #8344. No other files or hunks changed. No out-of-scope changes were found.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches

Generate unit tests (beta)

  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 45e42be4-7099-48d2-b201-1716380d11f3
Commits

Reviewing files that changed from the base of the PR and between b7a2c33 and a97ac6e.

Files selected for processing: 1

  • src/collections/news/2026/2026-01-21-docker-kanvas-kubernetes-dominance/index.mdx

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.

@leecalcote
leecalcote requested a review from yi-nuo426 November 8, 2026 17:11

@miacycle miacycle left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch, and thanks for the clear writeup tying it to #8344.

I verified the fix holds up: docker-kanvas-infoq.webp exists in the article directory, there is no .png by that name left, and darkthumbnail now matches thumbnail, so dark mode will stop pointing at a missing file.

One small check before this lands: could you confirm the dark-mode card for this post renders the thumbnail correctly in a local preview? The frontmatter change itself looks exactly right.

@leecalcote
leecalcote removed the request for review from yi-nuo426 November 8, 2026 17:29
@ayushbuilds-code

Copy link
Copy Markdown
Contributor Author

Thanks for the review @miacycle! Confirmed that the thumbnail asset resolves properly with darkthumbnail: ./docker-kanvas-infoq.webp without missing file errors in dark mode. Appreciate the review!

Signed-off-by: Ayush <ayushdungarwal3445@gmail.com>
@ayushbuilds-code
ayushbuilds-code force-pushed the fix/8344-docker-kanvas-news-darkthumbnail branch from a97ac6e to 3c20db4 Compare November 8, 2026 20:59
Signed-off-by: Ayush <ayushdungarwal3445@gmail.com>
@github-actions

github-actions Bot commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

Preview deployment for PR #8345 removed.

This PR preview was automatically pruned because we keep only the 3 most recently updated previews on GitHub Pages to stay within deployment size limits.

If needed, push a new commit to this PR to generate a fresh preview.

@miacycle miacycle left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for confirming the dark-mode render. I re-checked the head commit: the one-line change matches issue #8344 exactly, docker-kanvas-infoq.webp exists in the article directory, there is no .png left, and darkthumbnail now matches thumbnail. Checks are green. Approving.

@miacycle
miacycle merged commit 9179898 into layer5io:master Nov 9, 2026
5 checks passed
@ayushbuilds-code
ayushbuilds-code deleted the fix/8344-docker-kanvas-news-darkthumbnail branch November 9, 2026 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/news A noteworthy article, event, happening

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Broken darkthumbnail image path in Docker Kanvas news post

2 participants