Skip to content

Conversation

@lumirlumir
Copy link
Member

Prerequisites checklist

What is the purpose of this pull request?

In this PR, I've fixed CI failure caused by markdownlint.

Currently, the CI is failing in the main branch.

https://github.com/eslint/generator-eslint/actions/runs/18001203364/job/51210700196

image

This happened because of the update-readme automation, which was added in #225.

I've added <!-- markdownlint-disable MD033 --> and <!-- markdownlint-enable MD033 --> to allow HTML tags in a specific area.

What changes did you make? (Give an overview)

In this PR, I've fixed CI failure caused by markdownlint.

Related Issues

N/A

Is there anything you'd like reviewers to focus on?

N/A

@eslintbot eslintbot added this to Triage Sep 29, 2025
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Sep 29, 2025
@lumirlumir lumirlumir requested review from a team and aladdin-add September 29, 2025 08:32
Copy link

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Choose a reason for hiding this comment

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

+1 to disabling markdown linting on the auto-generated code.

@lumirlumir lumirlumir moved this from Needs Triage to Second Review Needed in Triage Sep 29, 2025
@nzakas
Copy link
Member

nzakas commented Sep 29, 2025

@JoshuaKGoldberg reminder: when you approve a PR, please either merge it or state the reason you didn't merge it.

Copy link
Member

@nzakas nzakas left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

@nzakas nzakas merged commit 8537e20 into main Sep 29, 2025
12 checks passed
@nzakas nzakas deleted the docs-fix-ci-failure-caused-by-markdownlint branch September 29, 2025 14:15
@github-project-automation github-project-automation bot moved this from Second Review Needed to Complete in Triage Sep 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

3 participants