feat(KLEF): Set up GitHub Actions for various code quality checks#1
Merged
isaacwallace123 merged 11 commits intomainfrom Mar 21, 2026
Merged
feat(KLEF): Set up GitHub Actions for various code quality checks#1isaacwallace123 merged 11 commits intomainfrom
isaacwallace123 merged 11 commits intomainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR migrates the repo away from the shared org-policies.yaml reusable workflow and replaces it with explicit, repo-local GitHub Actions workflows for code quality and docs checks.
Changes:
- Replaces the removed org-wide reusable workflow with standalone CI workflows (lint, typecheck, YAML lint, actionlint).
- Adds docs-focused quality gates (MDX linting via remark, spellchecking via cspell).
- Adds PR/process enforcement workflows (semantic PR title check and label policy enforcement).
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/org-policies.yaml |
Removes the legacy reusable workflow dependency. |
.github/workflows/actionlint.yml |
Adds workflow linting for GitHub Actions files. |
.github/workflows/yaml-lint.yml |
Adds yamllint across the repository. |
.github/workflows/typecheck.yml |
Adds TypeScript type checking in CI. |
.github/workflows/lint.yml |
Adds Next.js/ESLint linting in CI. |
.github/workflows/pr-title.yml |
Enforces semantic PR title formatting. |
.github/workflows/label-policy.yml |
Adds label sync + auto-label + required-label enforcement. |
.github/workflows/mdx-lint.yml |
Adds remark-based MDX/Markdown linting. |
.github/workflows/spellcheck.yml |
Adds cspell checks for .md/.mdx. |
.github/cspell.json |
Adds cspell configuration and project word allowlist. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…atures and integrate into CI.
…e checking, and action linting, including label definitions and auto-labeler configuration.
…pell checking, and Playwright E2E tests to the docs application.
… disable eslint's react/no-unescaped-entities rule.
… manual dispatch.
Reid910
approved these changes
Mar 21, 2026
Reid910
approved these changes
Mar 21, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request
Summary
org-policies.yamlreusable workflow (which pointed tokleffio/.github) with standalone inline CI checksRelated Issues
No related issue — part of org-wide CI migration away from reusable workflows in
.github.Changes
What's Included
org-policies.yaml(depended onkleffio/.githubreusable workflows)actionlint.yml— lints GitHub Actions workflow filesyaml-lint.yml— lints all YAML filestypecheck.yml— runstsc --noEmitto catch type errorslint.yml— runsnext lint(ESLint)pr-title.yml— enforces semantic PR title formatlabel-policy.yml— label sync → auto-label → require-label (chained withneeds)mdx-lint.yml— remark-based linting for MDX structure and headingsspellcheck.yml— cspell spellcheck across all.mdxand.mdfiles.github/cspell.json— cspell config with project-specific words whitelistedWhat's Not Included
package.jsonor any dependenciesTesting
How Was This Tested?
needsTest Coverage
Breaking Changes
Does this PR introduce breaking changes?
Security Considerations
Documentation
Does this PR require documentation updates?
UI/UX (If Applicable)
N/A
Pre-Merge Checklist
feat:,fix:,chore:,docs:,refactor:,test:)Reviewer Notes
The
label-policy.ymlworkflow runslabel-syncon both push and PR events so labels are guaranteed to exist beforeauto-labeltries to apply them. This is intentional.The
cspell.jsonwhitelist will likely need additions over time as more docs are written — contributors should add project-specific terms there rather than disabling the check.