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
58645: Remove grunt-jsvalidate from build process #4724
Conversation
@@ -1819,7 +1818,7 @@ module.exports = function(grunt) { | |||
if ( action !== 'deleted' ) { | |||
grunt.config( [ 'copy', 'dynamic-js', 'files' ], files ); | |||
} | |||
// For the webpack builds configure the jsvalidate task to only check those files build by webpack. | |||
// For the webpack builds configure the task to only check those files built by webpack. |
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.
I'm curious now that we're removing jsvalidate if this whole else if
could be removed or if it's still useful for some reason. I don't see that as a blocker to this patch though, so going to pre-approve.
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.
I wasn't certain so erred on the side of not changing the block, since it affects a non-validate-specific task.
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.
Committed in https://core.trac.wordpress.org/changeset/56247 |
Trac ticket: https://core.trac.wordpress.org/ticket/58645