Skip to content

Commit b25a914

Browse files
committed
Merge pull request #60 from loadsys/b/deploy-cleanup
Touch up the deploy script for Cake 3 project use.
2 parents 04d214f + 25d25cd commit b25a914

1 file changed

Lines changed: 10 additions & 5 deletions

File tree

deploy

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -73,6 +73,9 @@ else
7373
fi
7474

7575

76+
#@TODO: Maybe check for an executable `scripts/pre-deploy` project-specific script and run it? Abort this script if the pre-deploy exists non-zero?
77+
78+
7679
# Check for local changes that would cause a merge (without -f) to fail later.
7780
$BIN_DIR/git-localchanges
7881
if [ $? -gt 0 ]; then
@@ -111,7 +114,8 @@ elif [ "$DB_UPDATES_METHOD" = 'none' ]; then
111114
fi
112115

113116

114-
# This if case should really be looking to see if we're being asked to fast forward on the same branch, or switch to the HEAD of a different branch.
117+
# This if() should really be looking to see if we're being asked to fast
118+
# forward on the same branch, or switch to the HEAD of a different branch.
115119
# if [ "$ARG_NEWCOMMIT" = "origin/master" ]; then
116120
echo "## Merging upstream changes from: $ARG_NEWCOMMIT";
117121
git merge $ARG_NEWCOMMIT
@@ -131,11 +135,9 @@ if [ "$DB_UPDATES_METHOD" = 'migrations' ]; then
131135
$BIN_DIR/migrations
132136
fi
133137

134-
# Copy the correct config files into place for this environment.
135-
$BIN_DIR/set-configs $ARG_APPENV
136-
137-
138138
# Make sure any external packages are installed and available.
139+
# The current environment can export a COMPOSER_NODEV flag to disable
140+
# the installation of require-dev dependencies.
139141
composer install --no-interaction
140142

141143

@@ -164,5 +166,8 @@ if [ -n "${LOADSYS_EMAIL}" ]; then
164166
fi
165167

166168

169+
#@TODO: Maybe check for an executable `scripts/post-deploy` project-specific script and run it?
170+
171+
167172
echo "## Update process complete.";
168173

0 commit comments

Comments
 (0)