-
Notifications
You must be signed in to change notification settings - Fork 44
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 for Hardcoded Urls #28
Check for Hardcoded Urls #28
Conversation
<!-- No PHP short open tags allowed. --> | ||
<!-- Covers: https://github.com/Otto42/theme-check/blob/master/checks/phpshort.php --> | ||
<rule ref="Generic.PHP.DisallowShortOpenTag" /> | ||
|
||
|
||
<rule ref="WordPress-Theme" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you leave this out for the moment as it will make it easier to merge in the future as it may be added in another PR before hand?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
jrf walked me through this and included it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's fine. Listen to @jrfnl. She's the Boss! :P
563cd1d
to
6671823
Compare
Added exemption for theme and author urls by using the WordPress_AbstractThemeSniff class.
PR is not ready to be merged as we wish to add theme and author URLS to the whitelist. related #101 |
Closing for now, we may revisit this in another form in the future. |
This pull request looks at detecting hard coded urls in theme files.
close #14