Skip to content

ci: adopt the canonical org lint gate - #7

Merged
Bryan Fawcett (bryanfawcett) merged 1 commit into
mainfrom
ci/adopt-org-lint
Sep 11, 2026
Merged

Bryan Fawcett (bryanfawcett) merged 1 commit into
mainfrom
ci/adopt-org-lint

Conversation

@bryanfawcett

Copy link
Copy Markdown
Contributor

This repo published none of the five lint / * contexts the nyuchi org
ruleset requires on the default branch, so nothing could merge.
.github/workflows/lint.yml is now the canonical thin caller for
nyuchi/.github/.github/workflows/reusable-lint.yml. The job is named
lint and calls a reusable, so the checks publish as
lint / <called job> — exactly the five required strings.

Config files added, byte-identical to nyuchi/.github: .prettierrc,
.prettierignore, .markdownlint.jsonc, .yamllint.yaml, .editorconfig.

actionlint, JSON validity and yamllint were already clean. markdownlint
found two MD031 blanks-around-fences issues in packages/ui/BUILDING.md,
auto-fixed, and prettier reformatted two more markdown files. No hand
fixes were needed.

This repo carries a pnpm-lock.yaml, which the canonical .yamllint.yaml
now ignores alongside package-lock.json and yarn.lock — lockfiles are
generated, and pnpm copies registry deprecation notices into them
verbatim at lengths no one can rewrap.

Verified locally with CI's exact pinned versions (actionlint 1.7.12,
prettier 3.9.4, markdownlint-cli2 0.23.2, yamllint 1.38.0): all five
clean.

Co-Authored-By: Claude Opus 5 (1M context) noreply@anthropic.com

This repo published none of the five `lint / *` contexts the nyuchi org
ruleset requires on the default branch, so nothing could merge.
.github/workflows/lint.yml is now the canonical thin caller for
nyuchi/.github/.github/workflows/reusable-lint.yml. The job is named
`lint` and calls a reusable, so the checks publish as
`lint / <called job>` — exactly the five required strings.

Config files added, byte-identical to nyuchi/.github: .prettierrc,
.prettierignore, .markdownlint.jsonc, .yamllint.yaml, .editorconfig.

actionlint, JSON validity and yamllint were already clean. markdownlint
found two MD031 blanks-around-fences issues in packages/ui/BUILDING.md,
auto-fixed, and prettier reformatted two more markdown files. No hand
fixes were needed.

This repo carries a pnpm-lock.yaml, which the canonical .yamllint.yaml
now ignores alongside package-lock.json and yarn.lock — lockfiles are
generated, and pnpm copies registry deprecation notices into them
verbatim at lengths no one can rewrap.

Verified locally with CI's exact pinned versions (actionlint 1.7.12,
prettier 3.9.4, markdownlint-cli2 0.23.2, yamllint 1.38.0): all five
clean.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.

Tip: disable this comment in your organization's Code Review settings.

@bryanfawcett
Bryan Fawcett (bryanfawcett) merged commit 7bcd48f into main Sep 11, 2026
7 checks passed
@bryanfawcett
Bryan Fawcett (bryanfawcett) deleted the ci/adopt-org-lint branch September 11, 2026 19:02
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.

1 participant