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] Ajout d'info dans les message slack de PRs #74

Merged
merged 6 commits into from
May 12, 2021
Merged

Conversation

aceol
Copy link
Contributor

@aceol aceol commented May 10, 2021

🦄 Problème

On manque d'information sur les reviews affichés dans slack.

🤖 Solution

Afficher le statut des review (nombre d'approved, nombre de comments et de request change)

🌈 Remarques

On trie en plaçant en dernier les PR en cours de développement

💯 Pour tester

En local uniquement.
Lancer Pix-bot avec npm start.
Lancer ngrock avec ngrok http 3000.
Configurer la commande /pr-pix de l'environnement Slack Pix-Bot-Test avec l'adresse donnée par ngrok suivi de /slack/commands/pr-pix
Taper la commande /pr-pix cross-team
Verifier le message afficher

image

@aceol aceol changed the title Pr reviews info [Feature] Ajout d'info dans les message slack de PRs May 10, 2021
@aceol aceol force-pushed the pr-reviews-info branch 2 times, most recently from 91db292 to fbee370 Compare May 10, 2021 20:53
common/services/github.js Show resolved Hide resolved
common/services/github.js Outdated Show resolved Hide resolved
Copy link
Member

@VincentHardouin VincentHardouin left a comment

Choose a reason for hiding this comment

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

Sympa ! Est-ce que dans le fichier services/github il n'y aurait pas un petit BSR pour bien renommer les méthodes non exporter en les préfixant avec _ ?

@aceol
Copy link
Contributor Author

aceol commented May 12, 2021

Sympa ! Est-ce que dans le fichier services/github il n'y aurait pas un petit BSR pour bien renommer les méthodes non exporter en les préfixant avec _ ?

effectivement, ca m'a perturbé en arrivant sur ce fichier. Si tu veux t'en occuper n'hésites pas, sinon j'essaie de le faire dans la journée

@aceol aceol merged commit 339b72f into main May 12, 2021
@aceol aceol deleted the pr-reviews-info branch May 12, 2021 12:24
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants