Skip to content

Turn on ROBOT's label_whitespace check - #21

Open
hoodcm wants to merge 1 commit into
mainfrom
robot-label-whitespace
Open

hoodcm wants to merge 1 commit into
mainfrom
robot-label-whitespace

Conversation

@hoodcm

@hoodcm hoodcm commented Oct 1, 2026

Copy link
Copy Markdown
Collaborator

Part of #5.

Check Data checked Why
label_whitespace Labels A label has no leading or trailing space.

ROBOT's standard check, at its default level (ERROR). Nothing on main fails it, so no data changes. All tests pass.

@jclerman

jclerman commented Oct 1, 2026

Copy link
Copy Markdown
Collaborator

This looks like it will work as intended, though it is redundant with the intent of the previous rule. Still, no harm that I can see in adding it.

@jclerman jclerman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I confirmed via testing that this rule fires as expected and would prevent tests from passing in CI, so it should prevent bad values from getting merged into the main branch.

Approved.

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