Skip to content

Refactor markdownlint workflow#4999

Merged
LoopedBard3 merged 1 commit intodotnet:mainfrom
xtqqczze:refactor-markdownlintworkflow
Mar 19, 2026
Merged

Refactor markdownlint workflow#4999
LoopedBard3 merged 1 commit intodotnet:mainfrom
xtqqczze:refactor-markdownlintworkflow

Conversation

@xtqqczze
Copy link
Contributor

@xtqqczze xtqqczze commented Oct 1, 2025

  • Fix path to markdownlint.jsonc (broken in c3afd78)
  • Add permissions: contents: read to limit workflow permissions.
  • Switch Node version to lts/* for automatic long-term support updates.
  • Upgrade actions to latest releases.
  • Split the Markdown linting step into separate install and run steps.
  • Install markdownlint-cli with CI-friendly flags (--no-audit --no-fund).
  • Pin markdownlint-cli to version 0.45.0 to avoid potential build breaks when new versions are released.
  • Run markdownlint with an explicit rather than implicit configuration

@xtqqczze xtqqczze force-pushed the refactor-markdownlintworkflow branch from c9cbd4c to 28a10dd Compare October 1, 2025 15:34
@xtqqczze xtqqczze marked this pull request as ready for review October 1, 2025 15:38
@xtqqczze xtqqczze force-pushed the refactor-markdownlintworkflow branch from 28a10dd to ffc2039 Compare October 1, 2025 16:09
@xtqqczze xtqqczze force-pushed the refactor-markdownlintworkflow branch from ffc2039 to 02b9a0e Compare October 6, 2025 18:16
@xtqqczze xtqqczze force-pushed the refactor-markdownlintworkflow branch from 02b9a0e to ce433eb Compare February 27, 2026 19:47
* Fix path to `markdownlint.jsonc`
* Add `permissions: contents: read` to limit workflow permissions.
* Switch Node version to `lts/*` for automatic long-term support updates.
* Upgrade actions to latest releases.
* Split the Markdown linting step into separate install and run steps.
* Install `markdownlint-cli` with CI-friendly flags (`--no-audit` `--no-fund`).
* Pin `markdownlint-cli` to version `0.45.0` to avoid potential build breaks when new versions are released.
* Run `markdownlint` with an explicit rather than implicit configuration
@xtqqczze xtqqczze force-pushed the refactor-markdownlintworkflow branch from ce433eb to 4fb206d Compare March 2, 2026 18:12
@xtqqczze
Copy link
Contributor Author

@LoopedBard3 PTAL

Copy link
Member

@LoopedBard3 LoopedBard3 left a comment

Choose a reason for hiding this comment

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

Thanks for the PR!

@LoopedBard3 LoopedBard3 merged commit d76d845 into dotnet:main Mar 19, 2026
75 checks passed
@xtqqczze xtqqczze deleted the refactor-markdownlintworkflow branch March 21, 2026 20:02
@xtqqczze
Copy link
Contributor Author

@LoopedBard3 Thanks, I have a follow-up: #4998

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.

2 participants