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(valid-title): support mustMatch & mustNotMatch options #608

Merged
merged 1 commit into from Jul 5, 2020

Conversation

G-Rath
Copy link
Collaborator

@G-Rath G-Rath commented Jun 21, 2020

Closes #233
Closes #605

I implemented both mustMatch & mustNotMatch for the sake of completeness, since RegExp is not good at handling "not" matches.

@G-Rath G-Rath force-pushed the support-must-match-option branch from a70a4cc to 2e755dd Compare June 21, 2020 07:02
@G-Rath G-Rath requested a review from SimenB June 21, 2020 07:03
@G-Rath G-Rath force-pushed the support-must-match-option branch from 2e755dd to d845372 Compare June 22, 2020 21:46
@G-Rath G-Rath changed the title feat(valid-title): support mustMatch option feat(valid-title): support mustMatch & mustNotMatch options Jun 22, 2020
@G-Rath G-Rath force-pushed the support-must-match-option branch from d845372 to 6c34c88 Compare June 22, 2020 21:56
@G-Rath G-Rath merged commit 4c7207e into master Jul 5, 2020
@G-Rath G-Rath deleted the support-must-match-option branch July 5, 2020 09:59
github-actions bot pushed a commit that referenced this pull request Jul 5, 2020
# [23.18.0](v23.17.1...v23.18.0) (2020-07-05)

### Features

* **valid-title:** support `mustMatch` & `mustNotMatch` options ([#608](#608)) ([4c7207e](4c7207e)), closes [#233](#233)
@github-actions
Copy link

github-actions bot commented Jul 5, 2020

🎉 This PR is included in version 23.18.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@mkosir
Copy link

mkosir commented Aug 22, 2022

Thx for the PR, exactly what I need, would be great if we could also add an array of keywords for mustMatch and mustNotMatch, the same as we do for disallowedWords (we can always go with regex of course).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New rule request: title-matches-pattern
3 participants