Skip to content

Fix labels.exclude schema: object with name #870

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: main-enterprise
Choose a base branch
from
Open

Fix labels.exclude schema: object with name #870

wants to merge 1 commit into from

Conversation

frigus02
Copy link

@frigus02 frigus02 commented Aug 11, 2025 •
edited
Loading

The labels.exclude property takes an array of objects with name properties.

See usage:

safe-settings/lib/plugins/labels.js

Line 20 in 17e6d5c

this.exclude = exclude.filter(item => item?.name).map(item => new RegExp(item.name))

Possibly fixes #764

@frigus02 frigus02 force-pushed the fix-label-exclude-schema branch from 089318a to f9940c4 Compare August 11, 2025 12:57
@frigus02
Fix labels.exclude schema: object with name
1d214ce 
The labels.exclude property takes an array of objects with  properties.
@frigus02 frigus02 force-pushed the fix-label-exclude-schema branch from f9940c4 to 1d214ce Compare August 11, 2025 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Excluding labels will still show as deletion
1 participant
@frigus02