hello-jason / bedrock-deploy-to-wpengine Goto Github PK

I am considering removing the pieces of this script that build the website, and solely focusing on deploying to WP Engine.

Each site will most likely have different theme building needs, so you could setup your own build process then call this deploy script via gulp-shell, grunt-shell, or similar methods.

Fixed on branch hotfix/unnecessary-stdout. Needs merging.

• Should use exit 1 when failing to properly set script's exit status
  • This would allow, for example, ./wpedeploy.sh || printf "O NOE!\n"
  • wpedeploy.sh#L53
  • wpedeploy.sh#L59
  • wpedeploy.sh#69

The git repo for my project has this file structure:

.git
trellis
site

I placed the bash script inside site and setup the remotes for wpengine. However when I run the script, it pushes the trellis and site folders up to wpe. site/wp-content exists (as expected).

When I view wpe via sftp, I see the following folders at docroot:

site
trellis
wp-admin
wp-content
wp-includes
... other wp core files ...

This is likely because my development git repo isn't the bedrock repo, rather bedrock is within the site folder. Is it possible to deploy based on the relative path of the deploy script?

Figure out how to push up things in the mu-plugins directory as part of the deploy.

Related discussion here: https://discourse.roots.io/t/wpengine-bedrock-sage/3083/9

The script currently does not confirm if the user has these dependencies before running, and it will fail if they don't. Add these to the checks before it runs.

Determine if composer needs to run an update, install, or whatever rather than simply assuming that it needs to.

Change line 90 to:

echo -e '/*\n!wp-content/\nwp-content/uploads' > ./.gitignore

Single quotes let the bang character print properly.

The build process currently breaks with Sage v9.0.0-alpha3

Check if the current system has git push access to the WP Engine repo before branching and moving things. Otherwise, it leaves the user in a bad state on a new branch with files moved all around.