Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@caugner
Copy link
Contributor

@caugner caugner commented Oct 22, 2025

It is no longer used by any repo.

Description

Removes the markdown-lint workflow, because it is obsolete.

Motivation

It is no longer used by any repo.

Additional details

GitHub search: https://github.com/search?q=org%3Amdn+path%3A.github%2Fworkflows+%2Fmdn%5C%2Fworkflows%5C%2F.github%5C%2Fworkflows%5C%2Fmarkdown-lint.yml%2F&type=code

Related issues and pull requests

Part of mdn/fred#950.

@caugner caugner requested a review from a team as a code owner October 22, 2025 10:39
@caugner caugner requested review from LeoMcA and removed request for a team October 22, 2025 10:39
Copy link
Member

@LeoMcA LeoMcA left a comment

Choose a reason for hiding this comment

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

Approved, as the workflow doesn't appear to be used anywhere: but is there a reason content and translated-content have their own workflows instead of using this one?

@caugner
Copy link
Contributor Author

caugner commented Oct 22, 2025

Approved, as the workflow doesn't appear to be used anywhere: but is there a reason content and translated-content have their own workflows instead of using this one?

Generally, translated-content has a different setup. In particular, it requires a content checkout, so this workflow wouldn't work as is.

On the other hand, based on https://github.com/search?q=org%3Amdn+%2Fmarkdownlint-cli%2F+path%3A.github%2Fworkflows++NOT+is%3Aarchived&type=code, only translated-content seems to still have a markdown-lint workflow, whereas content runs it as part of the pr-check-lint_content workflow.

Overall, although content and translated-content already share most of the linting dependencies and scripts, the workflows seem to differ enough that it isn't straightforward to unify, and the added complexity just probably isn't worth it.

PS: If we had more repos with a single translated locale, it might be worth to extract those though.

@caugner caugner merged commit 612075d into main Oct 22, 2025
3 checks passed
@caugner caugner deleted the remove-markdown-lint-workflow branch October 22, 2025 17:04
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