Skip to content

skip compress images workflow on bot commits - #692

Open
AJaccP wants to merge 2 commits into
masterfrom
ajp/skip-bot-workflow-runs
Open

AJaccP wants to merge 2 commits into
masterfrom
ajp/skip-bot-workflow-runs

Conversation

@AJaccP

@AJaccP AJaccP commented Nov 14, 2026

Copy link
Copy Markdown
Collaborator

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@AJaccP
AJaccP requested a review from a team as a code owner November 14, 2026 15:54
@github-actions

github-actions Bot commented Nov 14, 2026 •

Copy link
Copy Markdown
Contributor

✅ Preview is ready!

PR preview ✅ Ready ✅ Ready
🔗 Preview https://CarletonComputerScienceSociety-website-build_preview-pr-692.surge.sh
📝 Commit2ef04eb
⏱️ Build time0.01s
📦 Size274.0 MB (no change) · 1805 files
🪵 LogsView logs
📱 MobileScan to open preview on mobile

↩️ Previous: ⚡️ 2ef04eb · CarletonComputerScienceSociety-website-build_preview-pr-692.surge.sh (open ↗) · 2026-11-14 15:58:48 UTC

🤖 Powered by surge-preview

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@AJaccP AJaccP changed the title skip workflow runs on image compression bot commits skip compress images workflow on bot commits Nov 14, 2026

@mpfaff mpfaff 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.

Looks good to me. I just want to note that I think if possible, we should compress the images in the repo side-by-side, and only include the compressed ones in the built website, so that we can improve the compression quality and file sizes across the board as that situation improves (i.e. webp is better than jpeg in many cases, and jpeg xl is better than webp in many cases and support for it is rolling out now).

This branch has not been deployed

No deployments
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.

2 participants