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

Déploiement coveralls #606

Closed
wants to merge 4 commits into from
Closed

Déploiement coveralls #606

wants to merge 4 commits into from

Conversation

firm1
Copy link
Contributor

@firm1 firm1 commented May 26, 2014

No description provided.

@GerardPaligot
Copy link
Member

Je profite de cette PR pour dire qu'il faudrait trouver un moyen pour
notifier tous les contributeurs que le fichier requirements.txt a changé,
non ?

2014-05-26 10:44 GMT+02:00 firm1 notifications@github.com:


You can merge this Pull Request by running

git pull https://github.com/firm1/zds-site init_travis_tests

Or view, comment on, or merge it at:

#606
Commit Summary

  • Update requirements.txt
  • Update .travis.yml

File Changes

Patch Links:


Reply to this email directly or view it on GitHubhttps://github.com//pull/606
.

@firm1
Copy link
Contributor Author

firm1 commented May 26, 2014

Je profite de cette PR pour dire qu'il faudrait trouver un moyen pour
notifier tous les contributeurs que le fichier requirements.txt a changé,
non ?

Je suis plutôt d'avis de demander aux devs de faire systématiquement un pip install --upgrade -r requirements.txt à chaque git pull.

@Taluu
Copy link
Contributor

Taluu commented May 26, 2014

Je sais pas si ajouter coveralls directement dans le requirements soit vraiment adapté, vu qu'on s'en sert que pour travis.

On pourrait pas juste ajouter une ligne pip install coveralls ou un truc du genre ?

@Alex-D
Copy link
Contributor

Alex-D commented May 26, 2014

+1 @Taluu : pensez à ma connexion du fond de la campagne les enfants !

@firm1
Copy link
Contributor Author

firm1 commented May 26, 2014

On pourrait pas juste ajouter une ligne pip install coveralls ou un truc du genre ?

pas con

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 4d290bc on firm1:init_travis_tests into * on zestedesavoir:dev*.

@firm1 firm1 closed this May 26, 2014
@firm1 firm1 deleted the init_travis_tests branch May 26, 2014 09:53
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

6 participants