- Notifications
You must be signed in to change notification settings - Fork397
library/xml.dom.minidom.po#1805
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
@sumit-158 como te comentaba en#1803 (comment), recuerda agregar en la descripción de este PR un comentario para que el issue asociado se cierre de forma automática. Puedes ver ejemplos en otros PRs de cómo se hace, debería ser tan simple como escribir "Closes #numerodelissue". Como te comentaba@cmaureir no es necesario cerrar el PR y abrir otro, puedes editar este mismo PR y todo bien. En el caso de comentarios sobre correcciones que sean necesarias, tampoco es necesario cerrar el PR y crear uno nuevo, solo basta con seguir haciendo commits en el branch con el cual creaste el PR y esos cambios se van a ver reflejados acá automáticamente. Y en el caso de las sugerencias de cambios específicos como la que te había hecho yo en la versión anterior de este PR, te debería haber aparecido un botón para hacer un commit con la sugerencia, lo que hace aún más fácil seguir iterando. Cualquier duda que tengas haz la pregunta, estamos para ayudar. |
This comment has been minimized.
This comment has been minimized.
modifiqué la descripción del issue, que no es 1805, es#1791 |
hay que quitar también el |
gracias por la traducción! |
Uh oh!
There was an error while loading.Please reload this page.
Closes#1791