Skip to content

Commit

Permalink
ci: Include root and component-parent into release parameter (#274)
Browse files Browse the repository at this point in the history
Details: currently, we have the `-am` parameter used in the component release build, 
as a side effect, if brings in unnecessary dependencies in the deploy list. 
To remove `-am`, we need to add the component parent project and root project 
into the `build.modules`
  • Loading branch information
ZheSun88 authored Oct 19, 2020
1 parent 29c3c5d commit b9d71c7
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions scripts/prepareDeploy.sh
Original file line number Diff line number Diff line change
Expand Up @@ -95,11 +95,11 @@ then
fi

echo "Deploying "`echo $modules | wc -w`" Modules from branch=$branch to profile=$profile"

build=vaadin-flow-components-shared
## '.' points to the root project, 'vaadin-flow-components-shared' has the dependencies for demo and tests
build=.,vaadin-flow-components-shared
for i in $modules
do
[ -d "$i" -o -d "$i-flow-parent" ] && build=$build,$i-flow-parent/$i-flow,$i-flow-parent/$i-testbench,$i-flow-parent/$i-flow-demo
[ -d "$i" -o -d "$i-flow-parent" ] && build=$build,$i-flow-parent,$i-flow-parent/$i-flow,$i-flow-parent/$i-testbench,$i-flow-parent/$i-flow-demo
done

## Inform TC about computed parameters
Expand Down

0 comments on commit b9d71c7

Please sign in to comment.