microsoft / microsoft/vscode-pull-request-github

Selecting `Approved` from the dropdown should not auto submit

Ouverte
#8,484 0 commentaires 1 réaction 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

feature-request
Langage dominant
TypeScript
Étoiles
2.6k
Forks
796
Merge moyen
1 j 4 h
PR mergées (30 j)
46

Description

Duplicate of https://github.com/microsoft/vscode-pull-request-github/issues/5300
Duplicate of https://github.com/microsoft/vscode-pull-request-github/issues/7028

I'd like to once again bring this issue up, because using a drop down to submit an action feels very unprecedented to me.

Like those other issues, I expect that hitting Approve here will change the blue Comment box from saying Comment to Approve, but will not submit the PR approval yet.

Image

To me that feels like a 1:1 mapping of the behavior of switching to the Approve radio button. Just clicking that radio button does not auto submit the PR.

Image

I often know I'm going to approve the PR, but haven't written my final approval comments in the box yet. On the github UI I'll change the radio button then go back and fill in the comments. I can't do this from the vs code extension and I end up double submitting so often 😢 .

I do like that if you widen the sidebar, then you see the 3 options inline in blue box form

Image

If you are unwilling to make the drop down be a pure "switcher", then at the very least if you could please style this drop down the same as the Create Merge Commit drop down, I think that would resolve a lot of my confusion.

Image

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Aucun fichier ni test n’est indiqué. Commencez par le menu déroulant de l’action de review de la pull request et comparez son comportement et son style avec ceux du menu déroulant « Create Merge Commit » ; le travail est terminé lorsque la sélection de « Approve » modifie l’action de review sans l’envoyer, afin de permettre la saisie de commentaires au préalable.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
typescript
Domaine
frontend
Type d'issue
Fonctionnalité
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.