-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Mentions in Changelog send notifications #1421
Comments
@jrjohnson yes will take a look at fixing this! |
Curiously, only the first part of the string up to the first code fence ( I've added a couple of tests in my fork that demonstrate the bug. If I have some spare time I'll take a look into fixing the implementation. |
Thanks @feelepxyz! I'll create a separate PR to handle an additional scenario with complex nested code fences too, to avoid false positives. |
Thank you! |
Seems to be a recurrence of #427 except for Changelog summary. As an example this PR rdohms/chainlink#36 mentions me in the Changelog section and I got notified about it
The text was updated successfully, but these errors were encountered: