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

Check if new notification events are documented #1

Open
zzak opened this issue Feb 4, 2023 · 1 comment
Open

Check if new notification events are documented #1

zzak opened this issue Feb 4, 2023 · 1 comment

Comments

@zzak
Copy link

zzak commented Feb 4, 2023

Similar to check-config-docs, I think it would be nice if we made sure new notifications were documented.

For example:
rails/rails#46603 (comment)

@skipkayhil
Copy link
Owner

This one may be tough to get high coverage on. Using my current method of AST parsing will be tough because there is so much variety in how hooks are defined. The ideal case looks something like this:

ActiveSupport::Notifications.instrument("process_action.action_dispatch", payload)

because we can easily extract the string, but in reality we have

I'll push up a script to get all of the easy ones once I refine it a bit more, but this may end up requiring a new approach 🤔

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

No branches or pull requests

2 participants