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

Upload latest build on success in Travis CI build #1173

Closed
wants to merge 5 commits into from
Closed

Upload latest build on success in Travis CI build #1173

wants to merge 5 commits into from

Conversation

pangratz
Copy link
Member

This is a WIP for using the upcoming feature of Travis CI to support encrypted environment variables inside the CI environment. With this feature, it is possible to have an up-to-date ember-latest.js in the repo's downloads section.

I created this PR so everybody can leave notes on the implementation.

As soon as this feature is available* in Travis CI, I'll update the PR accordingly.

*travis-ci/travis-core#76 (comment)

@travisbot
Copy link

This pull request fails (merged 32a3dd95 into 2b213df).

@travisbot
Copy link

This pull request fails (merged fa1aee3 into 2b213df).

@wagenet
Copy link
Member

wagenet commented Oct 8, 2012

@pangratz What's the status of this?

@pangratz
Copy link
Member Author

Taken from http://about.travis-ci.org/docs/user/build-configuration/

Please note that secure env variables are not available for pull requests. This is done due to security risk of exposing such information in submitted code. Everyone can submit a pull request and if an unencrypted variable is available there, it could be easily displayed.

So I'm closing this PR.

A possible solution for an up-to-date ember-master.js in the Downloads section is an app on Heroku which is informed on every commit on this repo, as described in #782.

@pangratz pangratz closed this Oct 18, 2012
@wagenet
Copy link
Member

wagenet commented Oct 19, 2012

@pangratz Darn. If you'd like to tackle such an app I'd definitely support that :)

@pangratz
Copy link
Member Author

Well, the basic app is available at https://github.com/pangratz/github-uploader. I'll update it so only pushes to the master branch trigger an upload ...

@wagenet
Copy link
Member

wagenet commented Oct 20, 2012

@pangratz, can you open a new ticket with the steps needed to make this operational?

@pangratz
Copy link
Member Author

Yup

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

Successfully merging this pull request may close these issues.

None yet

3 participants