Update reviewers-guide.rst #732
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
He añadido algo más de información. No he llegado a hacer todavía un par de cosas que me parecerían interesantes: - Añadir una sección sobre como traerte el código de una PR en concreto y hacer cambios si por algun motivo el traductor (autor de la PR) no puede hacerlos (x ej, pasar el powrap, etc) - Añadir un pequeño párrafo sobre cada uno de los 4 errores comunes técnicos que hemos puesto sobre cuales suelen ser las causas y cuales son las soluciones.
cmaureir
reviewed
Aug 29, 2020
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Manuel Kaufmann <humitos@gmail.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
clacri
commented
Aug 31, 2020
.overrides/reviewers-guide.rst
Outdated
| * “Needs work” genera en la línea anterior “#fuzzy” para que el traductor revise esa línea. Se puede usar cuando la traducción necesita revisión y en ese momento no puedes sugerir una traducción alternativa. | ||
|
|
||
| * PRs de más de 1000 líneas: Te sugerimos hacer un comentario al principio del PR para indicar hasta qué línea haz hecho el review y qué hay que revisar hasta ese hito. También si tú mismo vas a continuar la review, pero no puedes hacerla de una sola vez, es útil para saber donde retomar tu trabajo. | ||
| * El uso del checkbox “Needs work” en poedit genera en la línea anterior al párrafo un comentario “#fuzzy” para que el traductor revise esa línea. Se puede usar cuando la traducción necesita revisión y en ese momento no puedes sugerir una traducción alternativa. Ten en cuenta que los párrafos con esa marca no aparecerán traducidos en el build. Como revisor, también podrías incluir ese comentario tu mismo como sugerencia. |
Contributor
Author
There was a problem hiding this comment.
Suggested change
| * El uso del checkbox “Needs work” en poedit genera en la línea anterior al párrafo un comentario “#fuzzy” para que el traductor revise esa línea. Se puede usar cuando la traducción necesita revisión y en ese momento no puedes sugerir una traducción alternativa. Ten en cuenta que los párrafos con esa marca no aparecerán traducidos en el build. Como revisor, también podrías incluir ese comentario tu mismo como sugerencia. | |
| * El uso del checkbox "Needs work" en poedit genera en la línea anterior al párrafo un comentario "#fuzzy" para que el traductor revise esa línea. Se puede usar cuando la traducción necesita revisión y en ese momento no puedes sugerir una traducción alternativa. Ten en cuenta que los párrafos con esa marca no aparecerán traducidos en el build. Como revisor, también podrías incluir ese comentario tu mismo como sugerencia. |
clacri
commented
Aug 31, 2020
Contributor
Author
clacri
left a comment
clacri
left a comment
There was a problem hiding this comment.
intentando meter las sugerencias, github está raro con lo de aceptar y hacer commits xD
clacri
commented
Aug 31, 2020
clacri
commented
Aug 31, 2020
clacri
commented
Aug 31, 2020
clacri
commented
Aug 31, 2020
Contributor
Author
clacri
left a comment
clacri
left a comment
There was a problem hiding this comment.
creo que los he metido finalmente
cmaureir
approved these changes
Aug 31, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
He añadido algo más de información. No he llegado a hacer todavía un par de cosas que me parecerían interesantes: