Skip to content

Conversation

@mdjermanovic
Copy link
Member

Prerequisites checklist

What is the purpose of this pull request? (put an "X" next to an item)

[ ] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofix to a rule
[ ] Add a CLI option
[ ] Add something to the core
[x] Other, please explain:

Updates tools/update-readme.js to not cause formatting errors around sponsors placeholders:

https://github.com/eslint/eslint/actions/runs/13964654209/job/39092247760

What changes did you make? (Give an overview)

Added blank lines after <!--sponsorsstart--> and before <!--sponsorsend-->

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

@mdjermanovic mdjermanovic requested a review from a team as a code owner March 20, 2025 11:29
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Mar 20, 2025
@eslint-github-bot eslint-github-bot bot added the chore This change is not user-facing label Mar 20, 2025
@netlify
Copy link

netlify bot commented Mar 20, 2025

Deploy Preview for docs-eslint canceled.

Name Link
🔨 Latest commit 2e31c09
🔍 Latest deploy log https://app.netlify.com/sites/docs-eslint/deploys/67dbfc252498800008839679

Copy link
Member

@fasttime fasttime 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!

@fasttime fasttime merged commit 5228383 into main Mar 20, 2025
30 checks passed
@fasttime fasttime deleted the update-readme-formatting branch March 20, 2025 11:38
@github-project-automation github-project-automation bot moved this from Needs Triage to Complete in Triage Mar 20, 2025
@fasttime fasttime mentioned this pull request Mar 20, 2025
1 task
@fasttime fasttime added the accepted There is consensus among the team that this change meets the criteria for inclusion label Mar 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

accepted There is consensus among the team that this change meets the criteria for inclusion chore This change is not user-facing

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

3 participants