Skip to content

Conversation

@TKDev7
Copy link
Contributor

@TKDev7 TKDev7 commented Jun 23, 2025

Prerequisites checklist

What is the purpose of this pull request?

This pull request introduces a new option, checkMissingCells, to the table-column-count rule. The purpose of this addition is to provide users with more granular control over how the rule handles tables with missing cells, allowing for stricter or more flexible enforcement as needed.

What changes did you make? (Give an overview)

  • Added the checkMissingCells option to table-column-count
  • Updated rule logic, docs, and tests to support the new option

Related Issues

Fixes #422

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

@snitin315 snitin315 added this to Triage Jun 23, 2025
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Jun 23, 2025
@snitin315 snitin315 moved this from Needs Triage to Implementing in Triage Jun 23, 2025
snitin315
snitin315 previously approved these changes Jun 26, 2025
Copy link
Contributor

@snitin315 snitin315 left a comment

Choose a reason for hiding this comment

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

LGTM, would like another review before merge.

@snitin315 snitin315 moved this from Implementing to Second Review Needed in Triage Jun 26, 2025
@nzakas
Copy link
Member

nzakas commented Jul 11, 2025

@TKDev7 can you take a look at the merge conflict?

@lumirlumir this looks ready for you to re-review.

Copy link
Member

@lumirlumir lumirlumir left a comment

Choose a reason for hiding this comment

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

Sorry for the delay on my end.

I've responded to your previous comment. Everything else looks good to me.

Copy link
Member

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

@lumirlumir lumirlumir merged commit bf3f39b into eslint:main Jul 17, 2025
23 checks passed
@github-project-automation github-project-automation bot moved this from Second Review Needed to Complete in Triage Jul 17, 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.

Rule Change: Extend table-column-count to flag rows with fewer cells than header

4 participants