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

[FEATURE] Gérer le déploiement des review app sur scalingo #104

Merged
merged 3 commits into from
Apr 20, 2022

Conversation

francois2metz
Copy link
Contributor

@francois2metz francois2metz commented Apr 13, 2022

🦄 Problème

Nous souhaitons avoir la main sur le déploiement des review apps sur scalingo, en interdisant par défaut les forks typiquement.
De plus la logique de commentaire sur github est déplacé dans chaque repository.

🤖 Solution

Centraliser la gestion des reviews apps pour écouter les ouvertures de pull request sur l'organisation et réagir a partir de cela pour créer la review app d'une manière manuelle sur scalingo.

🌈 Remarques

La centralisation du post de commentaire sur github se fera dans un second temps.

💯 Pour tester

  1. Brancher un webhook sur l'organisation github et le brancher sur /github/webhook
  2. Ouvrir une pull-request sur 1024pix/pix
  3. Vérifier le retour du webhook et qu'il a bien déployé la PR.

@francois2metz francois2metz added cross-team Toutes les équipes de dev Tech Review Needed labels Apr 13, 2022
@francois2metz francois2metz self-assigned this Apr 13, 2022
@francois2metz francois2metz changed the title [FEATURE] [FEATURE] Gérer le déploiement des review apps sur scalingo Apr 13, 2022
@francois2metz francois2metz changed the title [FEATURE] Gérer le déploiement des review apps sur scalingo [FEATURE] Gérer le déploiement des review app sur scalingo Apr 13, 2022
@francois2metz francois2metz added team-captains This is your captain speaking and removed cross-team Toutes les équipes de dev labels Apr 13, 2022
@github-actions github-actions bot merged commit d9ef31a into main Apr 20, 2022
@github-actions github-actions bot deleted the deploy-ra branch April 20, 2022 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 Ready to Merge team-captains This is your captain speaking
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants