search

cnumr / ecoCode

Reduce the environmental footprint of your software applications with this cutting-edge sonarQube plugin
GNU General Public License v3.0
62 stars 48 forks source link

doc typo + upgrade installation #160

Closed dedece35 closed 2 years ago

dedece35 commented 2 years ago
dedece35 commented 2 years ago

hello @glalloue, merci pour la relecture. Comment ça marche pour le merge vers la "main" ? il faut un autre reviewer et le bouton de "merge" apparaitra ? ou c'est que je n'ai pas les droits de merger (car je ne vois pas le bouton pour "merger") ? thx.

glalloue commented 2 years ago

@dedece35 : je t'ai ajouté les droits d'écriture sur le repo. En théorie chaque PR doit passer par une revue des mainteneurs puis est "squash mergé" sur la main par eux. Mais vu que je pense que tu en ferras d'autres, après validation tu pourras faire le squash merge par toi même. Confirme moi si tu as bien accès au bouton mnt ;)

dedece35 commented 2 years ago

@glalloue, pas de souci pour de la relecture, c'est normal ... ça permet de checker qu'on ne fait pas de conneries et de montrer aux autres ce qui est fait. Je pense que même les mainteners devraient être relus. Je pense qu'il faut maintenir ce process de relecture. Par contre je ne vois pas le bouton :(

dedece35 commented 2 years ago

le statut de la PR est repassé en orange (alors qu'elle était verte). normal ? cf capture ci-jointe Capture d’écran 2022-10-26 à 17 49 58

dedece35 commented 2 years ago

(en effet, j'ai une autre PR dans les starting-blocs :p )

sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

dedece35 commented 2 years ago

@glalloue, c'est ok cette fois ... retour au vert et je viens de voir l'email suite aux changements de droits. Je viens d'accepeter l'invitation et du coup j'ai bien le bouton !

thx sir !

glalloue commented 2 years ago

ok le problème venait surement du fait que l'invitation n'avait pas été acceptée. Ouf, je ne savais plus quoi faire d'autre 😵‍💫