Gulp config and github actions#49
Merged
urchykli merged 4 commits intodevelopmentfrom May 14, 2021
Merged
Conversation
…e work with Local by flywheel
jnschrag
approved these changes
May 13, 2021
| WPE_SSH_KEY_PUBLIC: ${{ secrets.WPE_SSH_KEY_PUBLIC }} | ||
| # WPE_ENVIRONMENT: "production" | ||
| # WPE_LOCAL_BRANCH: "main" | ||
| # WPE_GIT_INCLUDE: ".github/wpe-git-include.txt" |
Contributor
There was a problem hiding this comment.
You might need to leave the Git include/exclude lines in there. You can remove the other two (environment & local branch).
urchykli
added a commit
that referenced
this pull request
May 14, 2021
* Delete MAMP PRO logo * Gulp config and github actions (#49) * Update gulpfile to use gulp config instead of config.json to make site work with Local by flywheel * Add github files for github actions deployment * Update github workflow to deploy to multiple environments * Git include and exclude files included in git deploy workflow * Define local branch names in github deploy script * Fix run syntax in GH deploy script * Change GH WPE deployment script to single site * Update secret key name and local branch in deployment script * Update wpengine-deploy.yml * Remove working directory in WPE deploy script * Revert to reconcile merge conflicts * Remove working directory in WPE deploy script * Gulp config points to correct sass src directory * Bug/aside images (#50) * Removed $.trim from tinymce.js on insert function. * Updatepackage lock * Remove node sass * Reinstall node sass * Remove cache npm from travis file as it's causing the build to error * Install node-gyp * Remove node-sass * In travis file change npm ci to npm install * Update package.json and run npm install Co-authored-by: MrBachChoi <55605733+MrBachChoi@users.noreply.github.com> * Update readme with correct gulp config file Co-authored-by: Jacque Schrag <jnschrag@users.noreply.github.com> Co-authored-by: MrBachChoi <55605733+MrBachChoi@users.noreply.github.com>
jdromero88
pushed a commit
that referenced
this pull request
Jan 8, 2025
* Update gulpfile to use gulp config instead of config.json to make site work with Local by flywheel * Add github files for github actions deployment * Update github workflow to deploy to multiple environments * Git include and exclude files included in git deploy workflow
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
In this PR I updated the
gulpfile.jsto look atgulp.config.jsfor thetheme_dirandsassconfigurations rather thanconfig.json. This allowed use of Local by flywheel.Additionally, I set up github actions and added the
.githubdirectory to the repo.