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

Deploy continuously #724

Merged
merged 5 commits into from
Apr 20, 2017
Merged

Deploy continuously #724

merged 5 commits into from
Apr 20, 2017

Conversation

cbenz
Copy link
Member

@cbenz cbenz commented Apr 4, 2017

Connected to openfisca/openfisca-ops#4

  • Amélioration technique.
  • Détails :
    • Mise en place du déploiement continu via la plateforme d'intégration continue Travis.

Ces changements :

  • Modifient des éléments non fonctionnels de ce dépôt.

@cbenz cbenz requested a review from fpagnoux April 4, 2017 12:49
.travis.yml Outdated
deploy:
- provider: script
skip_cleanup: true
script: ./travis-release-tag.sh
script:
- git tag "$(python setup.py --version)"
Copy link
Member

Choose a reason for hiding this comment

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

I'd be more confortable if this was tested on a test branch, with a dev version.

@@ -22,3 +30,7 @@ deploy:
distributions: "bdist_wheel"
on:
tags: true
- provider: script
script: ssh deploy-api@api.openfisca.fr
Copy link
Member

Choose a reason for hiding this comment

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

We should also do ssh deploy-new-api@api-test.openfisca.fr

Copy link
Member

@fpagnoux fpagnoux Apr 18, 2017

Choose a reason for hiding this comment

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

I guess ssh deploy-new-api@api.openfisca.fr is called by ssh deploy-api@api.openfisca.fr

@cbenz cbenz force-pushed the ci branch 2 times, most recently from f61ce21 to fa57feb Compare April 13, 2017 15:36
@fpagnoux fpagnoux merged commit 2ad17e1 into master Apr 20, 2017
@fpagnoux fpagnoux deleted the ci branch April 20, 2017 09:34
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