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

[FEATURE] Afficher les équipes et PRs concernées par une modif de fichier de config #295

Closed
wants to merge 3 commits into from
Closed

[FEATURE] Afficher les équipes et PRs concernées par une modif de fichier de config #295

wants to merge 3 commits into from

Conversation

yannbertrand
Copy link
Member

🦄 Problème

Lors d'une release, on prévient la personne qui a lancé la MER que le fichier de config a été modifié, ça concerne souvent des changements sur les variables d'environnements. Lors que ça arrive il faut :

  • soit demander aux équipes devs si ça les concerne
  • soit regarder dans l'historique qui est concerné et les alerter

🤖 Proposition

Je propose d'aller chercher l'info à la source :

  • On ajoute à la modal les liens vers les PRs qui ont modifié le fichier de config
  • On y ajoute aussi les labels d'équipes de ces PRs

🌈 Remarques

Je ne sais pas dans quoi je me suis lancé... Je suis à Montpellier il fait plus de 30 degrés je crois que je vais décéder. Voici ma dernière contribution.

💯 Pour tester

Je ne sais pas ?

@yannbertrand yannbertrand added cross-team Toutes les équipes de dev 👀 Tech Review Needed labels Aug 25, 2023
@yannbertrand yannbertrand self-assigned this Aug 25, 2023
@pix-bot-github
Copy link

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

@mickaelalibert mickaelalibert self-requested a review August 25, 2023 14:33
@yannbertrand yannbertrand changed the title [TECH] Afficher les équipes et PRs concernées par une modif de fichier de config [FEATURE] Afficher les équipes et PRs concernées par une modif de fichier de config Aug 25, 2023
yannbertrand
yannbertrand commented Aug 25, 2023
View reviewed changes
build/services/slack/surfaces/modals/publish-release/release-publication-confirmation.js
Comment on lines +25 to +30
'<https://github.com/' +
config.github.repoOwner +
'/' +
config.github.repoName +
'/pull/' +
pullRequestNumber +
Copy link
Member Author

Choose a reason for hiding this comment

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

Il existait pr.html_url mais je ne l'ai vu que trop tard... Du coup le champ n'est pas ramené depuis l'API GitHub jusque ici ^^"

@octo-topi
Copy link
Contributor

octo-topi commented Aug 28, 2023 •
edited
Loading

T'es un ouf Yann ! On a prévu de le faire côté Captains
https://1024pix.atlassian.net/browse/PIX-8514

@yannbertrand
Copy link
Member Author

T'es un ouf Yann ! On a prévu de le faire côté Captains 1024pix.atlassian.net/browse/PIX-8514

😅 vu la qualité du code vous devriez facilement pouvoir faire mieux :)

@octo-topi
Copy link
Contributor

Remplacée par #310

@octo-topi octo-topi closed this Sep 18, 2023
@mickaelalibert
Copy link
Contributor

Remplacée par #310

En partie, on va compléter avec des informations importantes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mickaelalibert mickaelalibert Awaiting requested review from mickaelalibert

Assignees

@yannbertrand yannbertrand

Labels
cross-team Toutes les équipes de dev 👀 Tech Review Needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@yannbertrand @pix-bot-github @octo-topi @mickaelalibert