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.

Sign up for GitHub

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

Jump to bottom

Fix #3571 : ajoute les dependances de production #3579

Merged
merged 1 commit into from
Jun 17, 2016
Merged

Fix #3571 : ajoute les dependances de production #3579

merged 1 commit into from
Jun 17, 2016

Conversation

gustavi
Copy link
Contributor

@gustavi gustavi commented May 3, 2016

Q R
Type de modification correction de bug
Ticket(s) (issue(s)) concerné(s) #3571

QA

  • Vérifier que les dépendances s'installent bien avec les instructions données dans la doc

@gustavi gustavi changed the title Fix #3571 : ajoute les dependances de producation Fix #3571 : ajoute les dependances de production May 3, 2016
@coveralls
Copy link

coveralls commented May 3, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 87.494% when pulling 5142813 on gustavi:fix-3571 into 4c6799f on zestedesavoir:dev.

@GerardPaligot
Copy link
Member

Il n'y a pas de script de déploiement à mettre à jour ici ?

@gustavi
Copy link
Contributor Author

gustavi commented May 3, 2016

Bien vu @GerardPaligot ! Ajouté.

@coveralls
Copy link

coveralls commented May 3, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 87.494% when pulling dc0449d on gustavi:fix-3571 into bbd193f on zestedesavoir:dev.

@gustavi
Copy link
Contributor Author

gustavi commented May 3, 2016

Tient npm qui fait des siennes, ça faisait longtemps.

@gustavi
Copy link
Contributor Author

gustavi commented Jun 16, 2016

UP QA

GerardPaligot
GerardPaligot reviewed Jun 17, 2016
View reviewed changes
requirements-prod.txt
@@ -0,0 +1,3 @@
gunicorn==19.4.5
MySQL-python==1.2.5
raven==5.14.0
Copy link
Member

Choose a reason for hiding this comment

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

Ces dépendances étaient installées comment avant sur la prod ? A la main ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oui, à la mano et sans verison spécifique :/

Copy link
Member

Choose a reason for hiding this comment

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

Ok ... :)

@GerardPaligot GerardPaligot merged commit a902374 into zestedesavoir:dev Jun 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Version 19
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@gustavi @coveralls @GerardPaligot