Skip to content

Add WebP-only linting rule - #155

Merged
samfreund merged 9 commits into
frcsoftware:mainfrom
httphypixelnet:feat/enforce-webp
Aug 7, 2026
Merged

Add WebP-only linting rule#155
samfreund merged 9 commits into
frcsoftware:mainfrom
httphypixelnet:feat/enforce-webp

Conversation

@httphypixelnet

@httphypixelnet httphypixelnet commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Description

Adds a lint-staged rule that fails commit if any image files not in WebP format are added.

Closes #116

Meta

Merge checklist:

@github-actions github-actions Bot added Curriculum material Curriculum materials, lessons for students, etc stage1 stage0 labels Jul 30, 2026
@github-actions

Copy link
Copy Markdown

🌐 Preview URL: https://pr-155.frcsoftware.pages.dev

@samfreund
samfreund marked this pull request as ready for review July 31, 2026 20:17

@samfreund samfreund left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't love the vale fix here, I'd prefer rewriting. I'll submit a separate PR for that, if you don't mind reverting that commit for this PR.

Comment thread scripts/images.lint.js Outdated
samfreund
samfreund previously approved these changes Aug 6, 2026
@samfreund
samfreund enabled auto-merge (squash) August 6, 2026 16:55
auto-merge was automatically disabled August 7, 2026 03:02

Head branch was pushed to by a user without write access

@httphypixelnet
httphypixelnet requested a review from a team as a code owner August 7, 2026 03:02
@github-actions github-actions Bot added the infra Any infrastructure for building the website or syncing files label Aug 7, 2026
@samfreund
samfreund enabled auto-merge (squash) August 7, 2026 03:02
@samfreund
samfreund merged commit 05a5ec5 into frcsoftware:main Aug 7, 2026
16 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Curriculum material Curriculum materials, lessons for students, etc infra Any infrastructure for building the website or syncing files stage0 stage1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Force adherence to image guidelines

2 participants