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

[TECH Ajout de commentaires de PR automatique contenant les urls des review apps. #189

Merged
merged 2 commits into from
Oct 13, 2020

Conversation

HEYGUL
Copy link
Contributor

@HEYGUL HEYGUL commented Oct 8, 2020

🦄 Problème

Il faut d'abord créer la Pull Request pour ensuite l'éditer et saisir manuellement les urls des review apps associées.

🤖 Solution

On utilise le même mécanisme que sur 1024pix/pix à savoir un script qui ajoute un commentaire automatique à la fin du build de l'application sur Scalingo.

🌈 Remarques

Il faut ajouter la variable REVIEW_APP=true sur le template de review app de pix-site et pix-site-pro.
Il faut également ajouter les credentials GitHub avec les variables GITHUB_USER_TOKEN et GITHUB_USER.

✨ Review App

https://site-pr189.review.pix.fr/
https://pro-pr189.review.pix.fr/

Copy link
Contributor

@bpetetot bpetetot left a comment

Choose a reason for hiding this comment

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

Apparement, il manque d'autres variables d'environnement sur le template de pix-site-review

FATAL: $GITHUB_USER is absent

Il doit également manquer $GITHUB_USER_TOKEN

@HEYGUL HEYGUL force-pushed the tech-add-review-app-urls-as-pr-comment branch 2 times, most recently from 0076919 to a635394 Compare October 8, 2020 16:17
@pix-service
Copy link

I'm deploying this PR to these urls:

Please check it out!

@bpetetot
Copy link
Contributor

bpetetot commented Oct 8, 2020

@HEYGUL Il faut ajouter les variables également sur le template de review app de pix-pro-review
Il y a un template par application (pix-site-review et pix-pro-review) (car 2 deploiements avec des variables d'env différentes).
PI: on a la même chose pour l'intégration. (pix-site-integration et pix-pro-integration)

@bpetetot
Copy link
Contributor

bpetetot commented Oct 8, 2020

On peut également supprimer le chapitre "Review APP" et les liens des app du template de PR.

@HEYGUL HEYGUL force-pushed the tech-add-review-app-urls-as-pr-comment branch from 06ea5ab to 00b273a Compare October 13, 2020 06:38
@HEYGUL HEYGUL merged commit a52f7e2 into dev Oct 13, 2020
@HEYGUL HEYGUL deleted the tech-add-review-app-urls-as-pr-comment branch October 13, 2020 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants