Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Ejecuta pospell con un comentario en el PR#1825

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

Closed
erickisos wants to merge18 commits intopython:3.10fromerickisos:github-actions

Conversation

erickisos
Copy link
Contributor

@erickisoserickisos commentedFeb 13, 2022
edited
Loading

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

Closes#1786

erickisosand others added9 commitsFebruary 13, 2022 12:01
* Agrego imagenes para el tutorial + instrucciones para generarlas* Quito los PNG porque pueden generarse a partir de los SVG
* Añade nuevo script para ordenar líneas de textoEl ordenamiento se hace usando el colaje de cadenas de texto del localeactual, 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 TRANSLATORSCuando nueva gente se agregue a sí misma a la lista de autores enTRANSLATORS, este nuevo paso en GitHub actions corroborará que elarchivo TRANSLATORS sigue estando ordenado alfabéticamente, y generaráun error de no ser éste el caso.Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>
* Usar 'lanzar' para 'raise'* Apply suggestions from code reviewCo-authored-by: rtobar <rtobar@icrar.org>
* 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óduloEl orden de las palabras no era del todo correcto.* Corrige "segundos bisiestos" por "segundos intercalares"Le da mas consistencia y exactitud (véasehttps://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ótonoLa palabra correcta es monótono (verhttps://es.wikipedia.org/wiki/Funci%C3%B3n_mon%C3%B3tona).* Corrige "programación" por "planificación"Creo que es mas correcto (verhttps://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
@rtobar
Copy link
Collaborator

No había visto esto, se ve súper!

Mi primer comentario es cambiar quizás el nombre del workflow y del archivo a algo que refleje mejor el hecho de que la acción ejecuta powrap (y hace un commit) al hacer un comentario ("powrap-on-demand" quizás). Se puede restringir a que la acción ocurra solamente cuando haya comentarios en un PR?

Por otro lado el chequeo de ortografía se podría quizas no realizar, ya que haría más lento el proceso. Además si la corrección falla, prevendría la corrección del largo de líneas de powrap, que es el cometido que se quiere lograr.

¿Quién necesita definir el GH token? ¿Este GH action corre bajo el usuario que crea el comentario me imagino?

@erickisos
Copy link
ContributorAuthor

@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 comentarpospell-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 apowrap-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
Copy link
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 comentarpospell-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 apowrap-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
Copy link
Collaborator

pospell-fix

@erickisos
Copy link
ContributorAuthor

@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
Copy link
ContributorAuthor

Cerrado en favor de#1836

@erickisoserickisos deleted the github-actions branchOctober 5, 2022 01:43
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@ezio-melottiezio-melottiAwaiting 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

[8]ページ先頭

©2009-2025 Movatter.jp