-
Notifications
You must be signed in to change notification settings - Fork 36
[DO_NOT_MERGE]chore: downstream check for all libraries in single job #621
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a few suggestions, otherwise LGTM
fail-fast: false | ||
matrix: | ||
java: [17] | ||
repo: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
any comments? how to update, auto update, ?
cd ${scriptDir}/.. | ||
|
||
# Make java core library artifacts available for 'mvn validate' at the bottom | ||
mvn install -DskipTests=true -Dmaven.javadoc.skip=true -Dgcloud.download.skip=true -B -V -q |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
consider adding -ntp
-fae
|
||
# run dependencies script | ||
cd .. | ||
mvn -Denforcer.skip=true clean install |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
consider adding -ntp
-fae
popd | ||
fi | ||
|
||
mvn -Denforcer.skip=true clean install |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
consider adding -ntp -fae
No description provided.