Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stop distribute script from uploading builds on new branch and tag #3

Open
brockboland opened this issue Sep 6, 2016 · 2 comments
Open

Comments

@brockboland
Copy link
Contributor

Our distribute.sh script checks if the build is running for a pull request and bails out if so, but it will continue on builds for new branches or tags (as well as merge builds, but that's the intended behavior).

This script should handle things differently in these cases, though I'm debating how differently. Distributing builds to HockeyApp and/or TestFlight for new branches and tags might be OK, but the release notes added to those should at least indicate which branch or tag the build is for (which can be accomplished using environment vars: https://docs.travis-ci.com/user/environment-variables/). Hockey builds include release notes (which currently indicate whether it's a staging or prod build). The pilot tool has a "changlog" option that's used for iTunes Connect builds, but I can't figure out if that's actually displayed anywhere in iTunes Connect.

Relevant environment variables:

  • TRAVIS_TAG
  • TRAVIS_BRANCH
@vokal-isaac
Copy link
Contributor

vokal-isaac commented Sep 6, 2016

n.b., it's possible to specify regex(es) for branches/tags to not build in .travis.yml, which we use to keep Travis from building the tags automatically generated by uploaded builds (https://github.com/vokal/Xcode-Template/blob/master/Vokal-Cocoa%20Touch%20Application%20Base.xctemplate/.travis.yml#L42-L44). I think it's also possible to specify only the tags/branches to include, rather than excluding some, and that we'd made the intentional decision that any tags/branches in the company shared repo should get built.

@brockboland
Copy link
Contributor Author

Letting Travis build for tags and branches is fine, I just don't think it should be distributing builds for these.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants