Skip to content

Limit the deployment only to the specific build job #250

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

Merged
merged 1 commit into from
Jan 9, 2019

Conversation

shinglyu
Copy link
Contributor

@shinglyu shinglyu commented Jan 7, 2019

Now if you push to the release branch, the deployment will run on every test suite. So all of them fails. We only want publish documentation to deploy to RustPython/docs and WASM online demo to release to RustPython/demo.

I can't seem to find a way to put the deployment as a post-build step for just one test suite, so I use this ugly environment variable to limit that.

Also here is a sample of a failed build: https://travis-ci.org/RustPython/RustPython/builds/476271838

@shinglyu
Copy link
Contributor Author

shinglyu commented Jan 7, 2019

will fix #249

.travis.yml Outdated
@@ -99,6 +101,7 @@ deploy:
keep-history: true
on:
branch: release
condition: $DEPLOY_DOC=ture
Copy link
Contributor

@windelbouwman windelbouwman Jan 7, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a typo on true here. Below as well.

.travis.yml Outdated
@@ -109,3 +112,4 @@ deploy:
keep-history: true
on:
branch: release
condition: $DEPLOY_DEMO=ture
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note the typo on ture

@shinglyu
Copy link
Contributor Author

shinglyu commented Jan 8, 2019

I also made a typo on branch name, jammer!

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.

2 participants