Skip to content

Configure Markdownlint to ignore eng/common files#2990

Merged
MihaZupan merged 3 commits intodarc-main-92774646-f62c-43c0-9a10-2bb2fbbb7a1ffrom
copilot/sub-pr-2989
Jan 21, 2026
Merged

Configure Markdownlint to ignore eng/common files#2990
MihaZupan merged 3 commits intodarc-main-92774646-f62c-43c0-9a10-2bb2fbbb7a1ffrom
copilot/sub-pr-2989

Conversation

Copy link
Contributor

Copilot AI commented Jan 21, 2026

The eng/common/ directory is managed by dotnet/arcade and should not be linted as part of this repository's checks.

Changes

  • Added markdownlintignore file in .github/workflows/ to exclude eng/common/ directory from Markdownlint validation
  • Updated the Markdownlint workflow to use --ignore-path option to reference the ignore file
  • Follows standard gitignore syntax for ignore patterns
# .github/workflows/markdownlintignore
eng/common/

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: MihaZupan <25307628+MihaZupan@users.noreply.github.com>
Copilot AI changed the title [WIP] Update dependencies from dotnet/arcade Configure Markdownlint to ignore eng/common files Jan 21, 2026
Copilot AI requested a review from MihaZupan January 21, 2026 15:52
@MihaZupan
Copy link
Member

@copilot Does this new file have to be at the root of the repo, or can it be placed next to the workflow file for markdownlint.
If it can, please move it under workflows. If not, are there other approaches?

Co-authored-by: MihaZupan <25307628+MihaZupan@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 21, 2026

@copilot Does this new file have to be at the root of the repo, or can it be placed next to the workflow file for markdownlint.
If it can, please move it under workflows. If not, are there other appr...

Yes, it can be placed next to the workflow file. Moved the ignore file to .github/workflows/markdownlintignore and updated the workflow to use --ignore-path option to reference it. (88e1f23)

@MihaZupan MihaZupan marked this pull request as ready for review January 21, 2026 16:15
@MihaZupan MihaZupan merged commit a634e6e into darc-main-92774646-f62c-43c0-9a10-2bb2fbbb7a1f Jan 21, 2026
2 of 6 checks passed
@MihaZupan MihaZupan deleted the copilot/sub-pr-2989 branch January 21, 2026 16:16
@MihaZupan MihaZupan added this to the YARP 3.0 milestone Feb 23, 2026
@MihaZupan MihaZupan added the Type: Infrastructure Infrastructure work that does not affect the actual shipping code label Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Infrastructure Infrastructure work that does not affect the actual shipping code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants