firka/flutter
firka/flutter
5
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Don't upload lcov to coveralls for pull requests (#8143)

Browse Source
This commit is contained in:
Adam Barth 2017-02-14 10:25:02 -08:00 committed by GitHub
parent c2eb73498d
commit 7f0b908ddc
2 changed files with 6 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
dev/bots/docs.sh
View File

@ -15,9 +15,7 @@ cp dev/docs/google2ed1af765c529f57.html dev/docs/doc
# Upload new API docs when on Travis and branch is master # Upload new API docs when on Travis and branch is master
if [ "$TRAVIS_PULL_REQUEST" = "false" ]; then if [ "$TRAVIS_PULL_REQUEST" = "false" ] && [ "$TRAVIS_BRANCH" = "master" ]; then
if [ "$TRAVIS_BRANCH" = "master" ]; then
cd dev/docs cd dev/docs
firebase deploy --project docs-flutter-io firebase deploy --project docs-flutter-io
fi fi
fi

6
dev/bots/travis_upload.sh
View File

@ -6,16 +6,16 @@ export PATH="$PWD/bin:$PWD/bin/cache/dart-sdk/bin:$PATH"
LCOV_FILE=./packages/flutter/coverage/lcov.info LCOV_FILE=./packages/flutter/coverage/lcov.info
if [ -n "$TRAVIS" ] && [ "$TRAVIS_PULL_REQUEST" == "false" ] && [ -a "$LCOV_FILE" ]; then if [ "$TRAVIS_PULL_REQUEST" = "false" ] && [ "$TRAVIS_BRANCH" = "master" ] && [ -f "$LCOV_FILE" ]; then
GSUTIL=$HOME/google-cloud-sdk/bin/gsutil GSUTIL=$HOME/google-cloud-sdk/bin/gsutil
GCLOUD=$HOME/google-cloud-sdk/bin/gcloud GCLOUD=$HOME/google-cloud-sdk/bin/gcloud
$GCLOUD auth activate-service-account --key-file ../gcloud_key_file.json $GCLOUD auth activate-service-account --key-file ../gcloud_key_file.json
STORAGE_URL=gs://flutter_infra/flutter/coverage/lcov.info STORAGE_URL=gs://flutter_infra/flutter/coverage/lcov.info
$GSUTIL cp $LCOV_FILE $STORAGE_URL $GSUTIL cp $LCOV_FILE $STORAGE_URL
(cd packages/flutter && coveralls-lcov coverage/lcov.info)
fi fi
# generate the API docs, upload them # generate the API docs, upload them
./dev/bots/docs.sh ./dev/bots/docs.sh
(cd packages/flutter && coveralls-lcov coverage/lcov.info)