-
-
Notifications
You must be signed in to change notification settings - Fork 28
Introduce conditionalTags #268
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
Conversation
Thank you for this PR.
Do we work on this package because we want users partially use it?
I was considering more 'controversial' rules that we might want to add (e.g., your suggestion in #197 or possibly enforcing DocBlocks for hooks). If we apply this approach to some rules, why not apply it to all rules?
more 'controversial' rules
Using static analysis for "'code'" (scripts) written for WordPress is 100% controversial.
What we three see okay for this package that must be used by all.
Using static analysis for "'code'" (scripts) written for WordPress is 100% controversial.
@IanDelMar Have you seen what others come up with before this package? Tragedy!
Allows rules to be disabled. Rules are enabled by default.