Skip to content

ci: add merge-blocking markdown format check - #109

Merged
kevmoo merged 2 commits into
mainfrom
markdown-ci
Sep 12, 2026
Merged

kevmoo merged 2 commits into
mainfrom
markdown-ci

Conversation

@kevmoo

@kevmoo kevmoo commented Sep 12, 2026 •

Copy link
Copy Markdown
Owner

Part of standardizing markdown formatting across the personal repos. This
repo's markdown is already byte-identical to the standard, so this PR adds
configuration only and changes no markdown whatsoever.

  • .prettierrc.json encodes the mdf standard (prose wrap at 80). Scoped to
    **/*.md via overrides so a stray prettier --write . cannot touch
    pubspec.yaml or workflow YAML. embeddedLanguageFormatting: off stops
    prettier rewriting fenced json/yaml examples.
  • markdown.yml pins prettier to the same version mise installs locally, and
    pins actions by SHA.

The job is named markdown because the job ID becomes the check-run context
that the branch ruleset matches on. The check is added to the ruleset only
after this lands, so no in-flight PR is stranded by a required context that no
workflow yet produces.

No version bump or changelog entry: this changes CI configuration only and
nothing in any published package.

Part of standardizing markdown formatting across the personal repos. This
repo's markdown is already byte-identical to the standard, so this PR adds
configuration only and changes no markdown whatsoever.

- `.prettierrc.json` encodes the `mdf` standard (prose wrap at 80). Scoped to
  `**/*.md` via `overrides` so a stray `prettier --write .` cannot touch
  `pubspec.yaml` or workflow YAML. `embeddedLanguageFormatting: off` stops
  prettier rewriting fenced json/yaml examples.
- `.prettierignore` excludes generated dartdoc output, which is untracked but
  would otherwise be caught by prettier's filesystem glob and make local runs
  disagree with CI.
- `markdown.yml` pins prettier to the same version `mise` installs locally, and
  pins actions by SHA.

The job is named `markdown` because the job ID becomes the check-run context
that the branch ruleset matches on. The check is added to the ruleset only
after this lands, so no in-flight PR is stranded by a required context that no
workflow yet produces.

No version bump or changelog entry: this changes CI configuration only and
nothing in any published package.
@github-actions

Copy link
Copy Markdown

📦 Dependency Lower-Bound Validation Summary

✅ git

  • SDK Floor: 3.9.0
  • Status: Clean
Dependency Declared Constraint Declared Floor Resolved Version Status
path ^1.8.0 1.8.0 1.8.0 Exact Floor
string_scanner ^1.4.1 1.4.1 1.4.1 Exact Floor

@github-actions

Copy link
Copy Markdown

📊 Cognitive Complexity Analysis

Net Delta: 0 | Added: 0 | Increased: 0 | Improved: 0 | Violations: 0

No modified Dart declarations detected.

The doc/api exclusion was speculative -- dartdoc output is never generated
into these repos, so the file only added noise to the canonical set.
@kevmoo
kevmoo merged commit f892686 into main Sep 12, 2026
14 checks passed
@kevmoo
kevmoo deleted the markdown-ci branch September 12, 2026 21:38
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