8000 chore: remove unused WebP OG image by dominicusin · Pull Request #428 · dominicusin/dominicusin.github.io · GitHub
[go: up one dir, main page]

Skip to content

chore: remove unused WebP OG image - #428

Merged
dominicusin merged 1 commit into
mainfrom
chore/remove-unused-webp
Sep 9, 2026
Merged

dominicusin merged 1 commit into
mainfrom
chore/remove-unused-webp

Conversation

@dominicusin
Copy link
Copy Markdown
Owner

Summary

  • Remove unused og-default.webp file (was not integrated into Hugo asset pipeline)

Why

The WebP variant was committed but never referenced in any template. Hugo's resources.Get doesn't auto-detect image variants — it requires explicit integration. Since the optimized PNG (66K) is already in use, the dead WebP file just adds repository bloat.

Validation

  • No references to og-default.webp in any template or content
  • hugo --gc --minify builds successfully

WebP variant was not integrated into Hugo asset pipeline —
the theme uses resources.Get which doesn't auto-detect variants.
Keeping only the optimized PNG (66K, -25% from original).
@bolt-new-by-stackblitz
Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Copilot AI commented Sep 9, 2026
Copy link
Copy Markdown

@dominicusin Thanks for sending me some feedback. Unfortunately, I hit an error while trying to use the custom Copilot setup steps configured for this repository. The error I am seeing is:

no supported setup-steps job found in your workflow file. Please ensure you have a job named copilot-setup-steps. For more details, see https://gh.io/copilot/actions-setup-steps

Once you or someone with the necessary access fixes the problem, please let me know in a comment and I'll try again. Thanks!

@vercel
vercel Bot commented Sep 9, 2026
Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
dominicusin-github-io Error Error Sep 9, 2026 3:44pm UTC
3 Skipped Deployments
Project Deployment Actions Updated
dominicusin-dominicusin.github.io Ignored Ignored Sep 9, 2026 3:44pm UTC
dominicusin-dominicusin.github.io-lw64 Ignored Ignored Sep 9, 2026 3:44pm UTC
dominicusin-github-io-5612 Ignored Ignored v0 Sep 9, 2026 3:44pm UTC

@github-actions github-actions Bot added the frontend JS/TS frontend modules label Sep 9, 2026
@github-actions
github-actions Bot commented Sep 9, 2026
Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@coderabbitai
coderabbitai Bot commented Sep 9, 2026
Copy link
Copy Markdown

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

💤 Files selected but had no reviewable changes (1)
  • assets/images/og-default.webp
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: b347a630-0d52-446f-9f97-9d26794a4646

📥 Commits

Reviewing files that changed from the base of the PR and between 23ce47d and 9237f82.

📒 Files selected for processing (1)
  • assets/images/og-default.webp

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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

Comment @coderabbitai help to get the list of available commands.

@qodo-code-review
Copy link
Copy Markdown

PR Summary by Qodo

Remove unused WebP Open Graph image

✨ Enhancement 🕐 Less than 5 minutes

Grey Divider

AI Description

• Removes the unreferenced WebP variant from the Hugo image assets.
• Retains the actively configured PNG fallback while reducing repository bloat.
High-Level Assessment

Deleting the unreferenced WebP asset is the appropriate approach because Hugo does not automatically select image variants and the configured PNG already serves the required social-image workflow. Explicitly integrating WebP was considered unnecessary without a current consumer.

@qodo-code-review
Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can commit Qodo's fix in one click with committable suggestions (GitHub & GitLab)

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@dominicusin
dominicusin merged commit f238838 into main Sep 9, 2026
22 of 25 checks passed
@dominicusin
dominicusin deleted the chore/remove-unused-webp branch September 9, 2026 15:59

This branch had an error being deployed

1 failed deployment
Preview – dominicusin-github-io — 9237f828 Deployed Sep 9, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend JS/TS frontend modules size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0