-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add support for rsync-ing a codebase from within the testing suite #332
Conversation
…ework into rsync-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.
You have a couple .DS_Store
files included and I found a couple other small things. Otherwise 👍
string $prefix, | ||
string $prefix_color, | ||
string $message, | ||
string $message_color = 'white', | ||
string $parent_classes = '', |
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.
do we need additional defaults?
); | ||
|
||
if ( 0 !== $retval ) { | ||
Utils::error( '🚨 Error installing rsyncing! Output from command:', 'Install Rsync' ); |
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.
slight grammar issue?
WP_PHP_BINARY, | ||
escapeshellarg( __DIR__ . '/install-wordpress.php' ), | ||
$multisite, | ||
// Utils::get_autoloader_path() |
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.
commented code
Add support for rsyncing a codebase (either a plugin/theme or wp-content) to within a WordPress installation -- all within the codebase. Removes the need for
rsync
-ing externally.