Skip to content
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.

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

feat: allow multiple title filters #182

Merged
merged 1 commit into from
Aug 29, 2024
Merged

feat: allow multiple title filters #182

merged 1 commit into from
Aug 29, 2024

Conversation

JoshuaKGoldberg
Copy link
Owner

@JoshuaKGoldberg JoshuaKGoldberg commented Aug 29, 2024

PR Checklist

Overview

Switches title on the CLI to multiple: true and internally to string[].

In theory I could have kept it singular by using a fancier regex. But that regex was getting overly complex.

💖

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

PR Compliance Checks Passed!

@JoshuaKGoldberg
Copy link
Owner Author

Lint failures are unrelated, must have been a bad automated dependency bump. Sigh. I also don't know why Dependabot recently started updating my things. I haven't enabled it or wanted it in a very long time, as I use Renovate instead...

@JoshuaKGoldberg JoshuaKGoldberg merged commit 3c0cb59 into main Aug 29, 2024
12 of 13 checks passed
@JoshuaKGoldberg JoshuaKGoldberg deleted the title-multiple branch August 29, 2024 18:50
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.

🚀 Feature: Expand title default to include build(dep-dev)
1 participant