Skip to content

chore: enable regex #7

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

stephansama
Copy link

@stephansama stephansama commented Aug 13, 2025

Initial checklist

  • I read the support docs
  • I read the contributing guide
  • I agree to follow the code of conduct
  • I searched issues and discussions and couldn’t find anything or linked relevant results below
  • I made sure the docs are up to date
  • I included tests (or that’s not needed)

Description of changes

Added the ability to use a string or regex with the plugin when matching comment titles

@github-actions github-actions bot added the 👋 phase/new Post is being triaged automatically label Aug 13, 2025

This comment has been minimized.

@github-actions github-actions bot added 🤞 phase/open Post is being triaged manually and removed 👋 phase/new Post is being triaged automatically labels Aug 13, 2025
@wooorm
Copy link
Member

wooorm commented Aug 14, 2025

Hey!

What’s the actual use case you are running into? What doesn’t work now but would work with this?

@stephansama
Copy link
Author

Hey!

What’s the actual use case you are running into? What doesn’t work now but would work with this?

Hey! thanks for looking at my PR! sorry i didnt add that information. so basically how i am using it is i am matching on comment patterns instead of just specific comment titles that way i can get comment meta data. i.e.

<!-- MATCH additional="arg" start -->

<!-- MATCH end -->

thats how i'm using it.
Thank you for reviewing the changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤞 phase/open Post is being triaged manually
Development

Successfully merging this pull request may close these issues.

2 participants