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

Retire des restes de Travis #6179

Merged
merged 6 commits into from
Oct 7, 2021
Merged

Retire des restes de Travis #6179

merged 6 commits into from
Oct 7, 2021

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Oct 1, 2021

On ne se sert plus de Travis pour la CI, mais il en restait des miettes. J'ai retiré tout ce que j'ai pu trouvé.

Fix #6175

Contrôle qualité

  • Vérifier que la CI passe toujours (normalement pas de soucis, c'est plus Travis ^^ )
  • Vérifier que le script d'installe fonctionne correctement (il y a eu des modifications sur l'affichage).

@Arnaud-D Arnaud-D added the C-DevelopmentEnv Amélioration de l'environnement de dev label Oct 1, 2021
Situphen
Situphen reviewed Oct 1, 2021
View reviewed changes
scripts/install_zds.sh Outdated Show resolved Hide resolved
@coveralls
Copy link

coveralls commented Oct 1, 2021 •
edited
Loading

Coverage Status

Coverage remained the same at 86.619% when pulling 7bbbbea on Arnaud-D:hasta_la_travista_baby into c8ba946 on zestedesavoir:dev.

@philippemilink philippemilink added the hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest label Oct 2, 2021
Situphen
Situphen approved these changes Oct 7, 2021
View reviewed changes
Copy link
Member

@Situphen Situphen left a comment

Choose a reason for hiding this comment

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

QA OK L'installation complète (make install-linux-full) fonctionne correctement, les messages d'informations s'affichent bien (bonne idée la mise en gras pour le titre de chaque étape) et la barre de défilement aussi pour les commandes wget.

@Situphen Situphen enabled auto-merge (rebase) October 7, 2021 15:18
@Situphen Situphen merged commit b547584 into zestedesavoir:dev Oct 7, 2021
@Situphen Situphen mentioned this pull request Oct 7, 2021
Merged
@Arnaud-D Arnaud-D deleted the hasta_la_travista_baby branch October 8, 2021 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Situphen Situphen Situphen approved these changes

Assignees
No one assigned
Labels
C-DevelopmentEnv Amélioration de l'environnement de dev hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest
Projects
Suivi des PR
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Retirer les dernier restes de Travis
4 participants
@Arnaud-D @coveralls @Situphen @philippemilink