Skip to content

Conversation

@caugner
Copy link
Contributor

@caugner caugner commented Feb 9, 2026

Description

Refines the pr-check-lint_content:

  1. Move only Markdown files, ignoring any non-Markdown files.
  2. Run the workflow on pull_request if the workflow itself gets changed.
  3. Lint Markdown files in the root directory.
  4. Separate tools into different steps, with diff after each step.

Motivation

  1. Avoid unintended side-effects of non-Markdown files on linting tools.
  2. Ensure workflow regressions are noticed before merging changes.
  3. Ensure workflow runs properly on PRs that only touch root Markdown files.
  4. Make it easier to identify which tool introduced a change.

Additional details

See this test run with example changes (92ef06c).

Related issues and pull requests

@github-actions github-actions bot added system [PR only] Infrastructure and configuration for the project size/s [PR only] 6-50 LoC changed labels Feb 9, 2026
- [Edge DevTools](https://learn.microsoft.com/en-us/microsoft-edge/devtools/landing/)
- : Documentation for the set of web-developer tools built into Edge.

Test No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

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

[mdn-linter] reported by reviewdog 🐶

Suggested change
Test
Test

@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2026

Preview URLs (1 page)

(comment last updated: 2026-02-09 20:58:25)

@caugner caugner force-pushed the pr-check-lint_content branch from ae7e6d2 to 3b29b0f Compare February 9, 2026 20:44
@caugner caugner force-pushed the pr-check-lint_content branch from 3b29b0f to 60b8124 Compare February 9, 2026 20:45
@github-actions github-actions bot added size/m [PR only] 51-500 LoC changed and removed size/s [PR only] 6-50 LoC changed labels Feb 9, 2026
@caugner caugner changed the title ci(pr-check-lint_content): move only Markdown files ci(pr-check-lint_content): refine workflow Feb 9, 2026
@caugner caugner marked this pull request as ready for review February 9, 2026 23:00
@caugner caugner requested a review from a team as a code owner February 9, 2026 23:00
@caugner caugner requested review from LeoMcA and removed request for a team February 9, 2026 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/m [PR only] 51-500 LoC changed system [PR only] Infrastructure and configuration for the project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant