Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

traduccion library/sys.monitoring.po #2710

Merged
merged 7 commits into from
Nov 3, 2023

Conversation

Alfareiza
Copy link
Contributor

@Alfareiza Alfareiza commented Oct 23, 2023

closes #2606

Copy link
Collaborator

@rtobar rtobar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sólo sugerencias por el momento basado en sphinx-lint

library/sys.monitoring.po Outdated Show resolved Hide resolved
library/sys.monitoring.po Outdated Show resolved Hide resolved
@rtobar
Copy link
Collaborator

rtobar commented Nov 1, 2023

Perdón @Alfareiza que no he podido volver acá... pero por mientras ojo que los tests aún están fallando

@Alfareiza
Copy link
Contributor Author

Perdón @Alfareiza que no he podido volver acá... pero por mientras ojo que los tests aún están fallando

Si, me di cuenta, solamente que ese error "PEP number must be a number from 0 to 9999" no sé como resolverlo. Me podrías ayudar?

@rtobar
Copy link
Collaborator

rtobar commented Nov 2, 2023

Perdón @Alfareiza que no he podido volver acá... pero por mientras ojo que los tests aún están fallando

Si, me di cuenta, solamente que ese error "PEP number must be a number from 0 to 9999" no sé como resolverlo. Me podrías ayudar?

Desafortunadamente pospell emite más mensajes de los que realmente constituyen los errores relacionados s ortografía. Para ver los verdaderos errores de pospell tienes que ir a las últimas líneas, ahí van a aparecer las palabras que no están reconocidas y en qué archivo (y creo que la línea también)

@Alfareiza
Copy link
Contributor Author

Ahora parece que está todo ok

Copy link
Collaborator

@rtobar rtobar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Un súper, súper buen trabajo como siempre @Alfareiza. Te dejo algunos comentarios, pero el grueso de la traducción está muy muy buena :)

library/sys.monitoring.po Outdated Show resolved Hide resolved
library/sys.monitoring.po Outdated Show resolved Hide resolved
library/sys.monitoring.po Outdated Show resolved Hide resolved
library/sys.monitoring.po Outdated Show resolved Hide resolved
library/sys.monitoring.po Outdated Show resolved Hide resolved
library/sys.monitoring.po Outdated Show resolved Hide resolved
library/sys.monitoring.po Outdated Show resolved Hide resolved
library/sys.monitoring.po Outdated Show resolved Hide resolved
library/sys.monitoring.po Outdated Show resolved Hide resolved
monitoreen
monitorean
monitoreados
stopiteration
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

supongo que sí, pero es necesario stopiteration? Hubiera pensado que no...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Inicialmente antes de las correcciones que me comentas, estaba dando un error de pospell con stopiteration.

@Alfareiza
Copy link
Contributor Author

Realicé todos los ajustes en un commit. Pensando en tu comodidad, te parece mejor así o prefieres que vaya comitando correción por correción a través de github?. Para mi (cuando hay muchos ajustes/correciones) no hay problema en como haga los commits.

Copy link
Collaborator

@rtobar rtobar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Un último detalle

library/sys.monitoring.po Outdated Show resolved Hide resolved
@rtobar
Copy link
Collaborator

rtobar commented Nov 3, 2023

Realicé todos los ajustes en un commit. Pensando en tu comodidad, te parece mejor así o prefieres que vaya comitando correción por correción a través de github?. Para mi (cuando hay muchos ajustes/correciones) no hay problema en como haga los commits.

Para mí tampoco hay lucha diferencia, yo uso la interfaz de GitHub para hacer las revisiones, así que no me importa mucho cuantos commits haya entre revisiones... Y al final del día hacernos un squash merge, así que aunque haya mil commits todos se reducen a uno solo

Copy link
Collaborator

@rtobar rtobar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gracias @Alfareiza 🎉

@rtobar rtobar merged commit 2240cd0 into python:3.12 Nov 3, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Translate library/sys.monitoring.po
2 participants