microsoft / microsoft/vscode-pull-request-github
Selecting `Approved` from the dropdown should not auto submit
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- TypeScript
- Estrellas
- 2.6k
- Forks
- 796
- Merge medio
- 1 d 4 h
- PR fusionados (30 d)
- 46
Descripción
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.
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.
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
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.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
No se mencionan archivos ni pruebas. Empieza por el menú desplegable de la acción de revisión del pull request y compara su comportamiento y estilo con el menú desplegable «Create Merge Commit»; se considera terminado cuando seleccionar «Approve» cambia la acción de revisión sin enviarla, lo que permite introducir comentarios primero.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- typescript
- Área
- frontend
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100