Skip to content

chore(ci): add schema validation to lint-staged config#538

Merged
KishonShrill merged 3 commits into
bettergovph:mainfrom
j4ckofalltrades:chore-schema-validation-pre-commit
May 19, 2026
Merged

chore(ci): add schema validation to lint-staged config#538
KishonShrill merged 3 commits into
bettergovph:mainfrom
j4ckofalltrades:chore-schema-validation-pre-commit

Conversation

@j4ckofalltrades
Copy link
Copy Markdown
Contributor

Schema validation now runs during the pre-commit hook so JSON validation errors can be caught early.

@DaijobuDes DaijobuDes added low priority This project can take it easy, no to long deadline. ci labels Nov 13, 2025
@j4ckofalltrades j4ckofalltrades force-pushed the chore-schema-validation-pre-commit branch 2 times, most recently from bfd2bf3 to 8b8f7ce Compare November 23, 2025 12:50
@j4ckofalltrades j4ckofalltrades force-pushed the chore-schema-validation-pre-commit branch from 2ff33a7 to bf026dc Compare December 6, 2025 07:51
@KishonShrill KishonShrill self-assigned this Dec 21, 2025
Schema validation now runs during the `pre-commit` hook so JSON
validation errors can be caught early.
@j4ckofalltrades j4ckofalltrades force-pushed the chore-schema-validation-pre-commit branch from bf026dc to 0ac6cd8 Compare May 19, 2026 14:54
@KishonShrill
Copy link
Copy Markdown
Member

@j4ckofalltrades do you mind if you can do an npm update too? Just so we can quiet/lessen the 7 vulnerabilities.

Upgrade vite 5→7 to resolve GHSA-67mh-4wv8-2f99 (esbuild dev server CORS
bypass). Add ws@8.20.1 override to resolve GHSA-58qx-3vcg-4xpx
(uninitialized memory disclosure in miniflare's pinned ws@8.18.0).
Copy link
Copy Markdown
Member

@KishonShrill KishonShrill left a comment

Choose a reason for hiding this comment

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

Alright, this looks good

@KishonShrill KishonShrill merged commit dc16490 into bettergovph:main May 19, 2026
9 checks passed
@j4ckofalltrades j4ckofalltrades deleted the chore-schema-validation-pre-commit branch May 19, 2026 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci low priority This project can take it easy, no to long deadline.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants