ci: add merge-blocking markdown format check - #108
Merged
Merged
Conversation
Pilot for standardizing markdown formatting across the personal repos. This repo was chosen first because its markdown is already byte-identical to the standard, so the PR validates the CI wiring against a zero-line formatting diff -- if it goes red, the workflow is unambiguously at fault. - `.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 to match the existing convention in `ci.yml`. 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 the published package.
📦 Dependency Lower-Bound Validation Summary✅
|
| Dependency | Declared Constraint | Declared Floor | Resolved Version | Status |
|---|---|---|---|---|
args |
^2.0.0 |
2.0.0 |
2.0.0 |
Exact Floor |
build_cli_annotations |
^2.0.0 |
2.0.0 |
2.0.0 |
Exact Floor |
shelf |
^1.0.0 |
1.0.0 |
1.0.0 |
Exact Floor |
shelf_static |
^1.0.0 |
1.0.0 |
1.0.0 |
Exact Floor |
📊 Cognitive Complexity AnalysisNet Delta: 0 | Added: 0 | Increased: 0 | Improved: 0 | Violations: 0 No modified Dart declarations detected. |
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.
Pilot for standardizing markdown formatting across the personal repos. This
repo was chosen first because its markdown is already byte-identical to the
standard, so the PR validates the CI wiring against a zero-line formatting
diff -- if it goes red, the workflow is unambiguously at fault.
.prettierrc.jsonencodes themdfstandard (prose wrap at 80). Scoped to**/*.mdviaoverridesso a strayprettier --write .cannot touchpubspec.yamlor workflow YAML.embeddedLanguageFormatting: offstopsprettier rewriting fenced json/yaml examples.
.prettierignoreexcludes generated dartdoc output, which is untracked butwould otherwise be caught by prettier's filesystem glob and make local runs
disagree with CI.
markdown.ymlpins prettier to the same versionmiseinstalls locally, andpins actions by SHA to match the existing convention in
ci.yml.The job is named
markdownbecause the job ID becomes the check-run contextthat 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 the published package.