Skip to content

Commit

Permalink
Remove composer dependency listing for built branches (#14900)
Browse files Browse the repository at this point in the history
* Remove composer dependencies listing for built branches

Ref: #13497

* Improve removing composer packages listing script

Simplifying by using composer regexp capability

* Cleaning up the composer edition in the build script
  • Loading branch information
LeoColomb authored Mar 18, 2020
1 parent 29426e4 commit 22eb790
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions tools/build-release-branch.sh
Original file line number Diff line number Diff line change
Expand Up @@ -261,6 +261,12 @@ yarn cache clean
COMPOSER_MIRROR_PATH_REPOS=1 yarn run build-production
echo "Done"

echo "Remove composer dependencies listing"
# Workaround for https://github.com/Automattic/jetpack/issues/13497
# TODO: Keep dependencies away from built branches and fetch them before publishing on WP-org plugins repository.
composer remove "automattic/*" --no-update
echo "Done"

# Prep a home to drop our new files in. Just make it in /tmp so we can start fresh each time.
rm -rf TMP_REMOTE_BUILT_VERSION
rm -rf TMP_LOCAL_BUILT_VERSION
Expand Down Expand Up @@ -298,6 +304,7 @@ echo "Done! Branch $BUILD_TARGET has been updated."

echo "Cleaning up the mess"
cd $DIR
git checkout -- composer.json
rm -rf TMP_REMOTE_BUILT_VERSION
rm -rf TMP_LOCAL_BUILT_VERSION
echo "All clean!"

0 comments on commit 22eb790

Please sign in to comment.