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] Envoyer le commentaire de déploiement des RAs sur les PRs quelque soit leur état #511

Merged
merged 1 commit into from
Jan 28, 2025

Conversation

yannbertrand
Copy link
Member

🥞 Problème

Lors de la création des Reviews App d'une PR, on ajoute un commentaire GitHub pour donner les URLs de ces environnements temporaires. Ce commentaire est conditionné au bon déploiement de la RA. Ce déploiement est relativement instable, il arrive donc que le commentaire ne soit jamais envoyé.

🥓 Proposition

Essayer de ne pas lier le commentaire au bon déploiement de la RA.

🧃 Remarques

RAS

😋 Pour tester

Je ne sais pas

@yannbertrand yannbertrand added the cross-team Toutes les équipes de dev label Jan 28, 2025
@yannbertrand yannbertrand self-assigned this Jan 28, 2025
@pix-bot-github
Copy link

Une fois l'application déployée, elle sera accessible à cette adresse https://bot-pr511.review.pix.fr
Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-bot-review-pr511/environment

@yannbertrand yannbertrand changed the title [FEATURE] [FEATURE] Envoyer le commentaire de déploiement des RAs sur les PRs quelque soit l'état de leur déploiement Jan 28, 2025
@yannbertrand yannbertrand changed the title [FEATURE] Envoyer le commentaire de déploiement des RAs sur les PRs quelque soit l'état de leur déploiement [FEATURE] Envoyer le commentaire de déploiement des RAs sur les PRs quelque soit leur état Jan 28, 2025
@nlepage nlepage force-pushed the early-comment-ra-urls branch from e13e001 to 96b1a91 Compare January 28, 2025 15:07
@pix-service-auto-merge pix-service-auto-merge merged commit cef6bc7 into main Jan 28, 2025
5 of 6 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the early-comment-ra-urls branch January 28, 2025 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-team Toutes les équipes de dev 🚀 Ready to Merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants