Ejecuta pospell con un comentario en el PR by erickisos · Pull Request #1825 · python/python-docs-es · GitHub
Skip to content

Ejecuta pospell con un comentario en el PR - #1825

Closed
erickisos wants to merge 18 commits into
python:3.10from
erickisos:github-actions
Closed

Ejecuta pospell con un comentario en el PR#1825
erickisos wants to merge 18 commits into
python:3.10from
erickisos:github-actions

Conversation

@erickisos

@erickisos erickisos commented Feb 13, 2022 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Para poder habilitarlo, necesitamos definir un GithubToken en el repositorio.

Closes #1786

erickisos and others added 9 commits February 13, 2022 12:01
@erickisos
wip: github action to run pospell
9f36e72
@cacrespo
corrige y añade traductores faltantes
fe540b9
@raulcd
Merge pull request python#1827 from cacrespo/fix_translators
04cd006 
corrige y añade traductores faltantes
@mmmarcos
logo python-docs-es + instrucciones de cómo generarlo (python#1828)
bbfed35 
* Agrego imagenes para el tutorial + instrucciones para generarlas
* Quito los PNG porque pueden generarse a partir de los SVG
@rtobar
Chequea orden en TRANSLATORS (python#1830)
29d216b 
* Añade nuevo script para ordenar líneas de texto

El ordenamiento se hace usando el colaje de cadenas de texto del locale
actual, y sobre stdin, haciendo la herramienta lo más flexible posible.

Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>

* Agrega paso a CI para chequear orden en TRANSLATORS

Cuando nueva gente se agregue a sí misma a la lista de autores en
TRANSLATORS, este nuevo paso en GitHub actions corroborará que el
archivo TRANSLATORS sigue estando ordenado alfabéticamente, y generará
un error de no ser éste el caso.

Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>
@cacrespo
nombre de directorio adecuado (python#1829)
23e34ff
@cmaureir @rtobar
Usar 'lanzar' para 'raise' (python#1816)
a7fbd71 
* Usar 'lanzar' para 'raise'

* Apply suggestions from code review

Co-authored-by: rtobar <rtobar@icrar.org>
@gio8tisu
Varias correcciones en el módulo time (python#1831)
ef2d567 
* Corrige título del módulo
'Time access' se refiere a 'acceso a (funcionalidades relacionadas con)
tiempo' no a 'tiempo de acceso'.
* Aclara la descripcion de disponibilidad del módulo
El orden de las palabras no era del todo correcto.
* Corrige "segundos bisiestos" por "segundos intercalares"
Le da mas consistencia y exactitud (véase
https://es.wikipedia.org/wiki/Segundo_intercalar).
* Corrige errores menores
* "mucho en el futuro" por "muy adelante en el futuro"
* Varios articulos
* Palabras sin traducir
* Corrige monotónico por monótono
La palabra correcta es monótono (ver
https://es.wikipedia.org/wiki/Funci%C3%B3n_mon%C3%B3tona).
* Corrige "programación" por "planificación"
Creo que es mas correcto (ver https://es.wikipedia.org/wiki/Planificador).
* Corrige "sentido" por "significado"
* Usar "lanzar" para "raise"
* Corrige "julio" por "juliano"
* Fix wrapping
* Añado preposicion en frase de funciones C
* Corrige wrapping manualmente
@ezio-melotti
Fix built-in Sphinx substitutions. (python#1834)
6713fc1
@rtobar

rtobar commented Oct 3, 2022

Copy link
Copy Markdown
Collaborator

@erickisos

erickisos commented Oct 3, 2022

Copy link
Copy Markdown
Collaborator Author

@rtobar gracias por dejar tus dudas, todas son tremendas preguntas, así que déjame ver si las puedo responder de a poquillo.

  • Se puede restringir a que la acción ocurra solamente cuando haya comentarios en un PR?: Según veo, lo que estaba haciendo en su momento era que al comentar pospell-fix en un PR se ejecutara el workflow, es decir, solo si el comentario existe y específicamente diciendo eso (Pero podemos evaluar el comando en sí, o cambiarlo a powrap-on-demand como mencionas.
  • Quién necesita definir el GH token?: Después de revisar en otros lados, creo que no es necesario porque el accion lo toma automático del repositorio, pero vale la pena probarlo solo para estar seguros.
  • Este GH action corre bajo el usuario que crea el comentario...?: Según entiendo, no, los GH actions tienen su propia "entidad" por así decirlo, y generan commits a su propio nombre, solo dependen del permiso que les puede dar un GHToken.

@rtobar

rtobar commented Oct 3, 2022

Copy link
Copy Markdown
Collaborator
  • Se puede restringir a que la acción ocurra solamente cuando haya comentarios en un PR?: Según veo, lo que estaba haciendo en su momento era que al comentar pospell-fix en un PR se ejecutara el workflow, es decir, solo si el comentario existe y específicamente diciendo eso (Pero podemos evaluar el comando en sí, o cambiarlo a powrap-on-demand como mencionas.

Perdón que no expresé bien mi pregunta: mi duda era si acaso tal como está implementando la acción correría también si alguien comenta en un issue en vez de un PR; en ese contexto quería clarificar si acaso se puede restringir a sólo PRs, o si acaso el trigger "on.issue_comment" ya aplica implícitamente sólo a PRs.

El comando en si quizás puede ser "run-powrap" o algo similar,

  • Quién necesita definir el GH token?: Después de revisar en otros lados, creo que no es necesario porque el accion lo toma automático del repositorio, pero vale la pena probarlo solo para estar seguros.

Mmmm... Debería ser posible revisar si comento en este PR, no? Lo voy a intentar

  • Este GH action corre bajo el usuario que crea el comentario...?: Según entiendo, no, los GH actions tienen su propia "entidad" por así decirlo, y generan commits a su propio nombre, solo dependen del permiso que les puede dar un GHToken.

Vale, veamos qué pasa ahora que lo tire a probar.

@rtobar

rtobar commented Oct 3, 2022

Copy link
Copy Markdown
Collaborator

pospell-fix

ezio-melotti
ezio-melotti reviewed Oct 3, 2022
View reviewed changes
Comment thread .github/workflows/pospell.yml Outdated
Comment thread .github/workflows/pospell.yml Outdated
Comment thread .github/workflows/pospell.yml
Comment thread .github/workflows/pospell.yml
Comment thread .github/workflows/pospell.yml
Comment thread .github/workflows/pospell.yml Outdated
@erickisos

erickisos commented Oct 4, 2022

Copy link
Copy Markdown
Collaborator Author

@rtobar just in case, es posible que no corra en el PR dado que está en otro repo, pero puedo probar mezclando en mi propio fork para ejecutarlo

@erickisos
erickisos requested a review from ezio-melotti October 4, 2022 05:54
@erickisos

erickisos commented Oct 5, 2022

Copy link
Copy Markdown
Collaborator Author

@erickisos erickisos closed this Oct 5, 2022
@erickisos
erickisos deleted the github-actions branch October 5, 2022 01:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ezio-melotti ezio-melotti Awaiting requested review from ezio-melotti

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Robot powrap en GitHub

8 participants

@erickisos @rtobar @ezio-melotti @cacrespo @raulcd @mmmarcos @cmaureir @gio8tisu