Github Action para ejecutar powrap fix con un comentario#1836
Github Action para ejecutar powrap fix con un comentario#1836erickisos wants to merge 35 commits into
Conversation
Sorry, something went wrong.
Co-authored-by: Ezio Melotti <ezio.melotti@gmail.com>
Add Github action to run pospell on a comment
Update check_spell.py
React to comment
Update pospell-on-demand.yml
add cache apt action
Este cambio no está relacionado al PR, así que no hace sentido tenerlo acá.
|
Acá está una versión que saqué después de probar en mi fork @rtobar @ezio-melotti 🥳 |
Sorry, something went wrong.
|
pospell-fix |
Sorry, something went wrong.
ezio-melotti
left a comment
ezio-melotti
left a comment
There was a problem hiding this comment.
Algunas sugerencias sobre un nuevo comando pospell-check que solo dice si la PR es buena sin hacer el commit.
Tambien podrias ejecutar el check automaticamente cuando la PR es creada/modificada, y solo haber el comando pospell-fix para hacer el commit.
Sorry, something went wrong.
erickisos
requested review from
ezio-melotti and
rtobar
and removed request for
rtobar
October 5, 2022 04:40
|
Ya hice varios de los cambios sugeridos, pero lamentablemente creo que no podemos ejecutarlo hasta que le demos merge en este repo 😞 De todas maneras por si las moscas, voy a crear un PR en mi fork, para probar la ejecución. |
Sorry, something went wrong.
|
Tras los cambios, en este PR (erickisos#6) ejecuté el comando y podemos ver los resultados acá |
Sorry, something went wrong.
rollback to previous commit action
Add branch name to action-js
Update execute-powrap.yml
Update execute-powrap.yml
|
Desafortunadamente me he dado cuenta de que el commit action no encuentra cambios tras ejecutar powrap, no estoy seguro de por qué 😢 |
Sorry, something went wrong.
cmaureir
added
the
hacktoberfest-accepted
mantainer-approved contribution to hacktoberfest
label
Oct 25, 2022
rtobar
left a comment
rtobar
left a comment
There was a problem hiding this comment.
Perdón @erickisos por tomarme tanto tiempo en responder pero andaba de vacaciones y sin mucho tiempo para ver otros temas.
Veo que la mayoría de los comentarios ya los tomaste en cuenta, y la acción ahora sólo ejecuta powrap como era la intención original. Estuve viendo los ejemplos de ejecución que pusiste, y otros más, y encontré otro par de problemas que hay que arreglar, pero en general ser está viendo súper bien 👍
Sorry, something went wrong.
|
En #1786 (comment) comenté porqué creo que esto puede ser un problema, y propuse una solución "inofensiva" a primera vista. |
Sorry, something went wrong.
|
Este PR lleva un tiempo sin actualizaciones. Vamos a pedir a un admin de nuestro equipo que decida si alguien más puede finalizarlo o si tenemos que cerrarlo.\nPor favor, avisanos en caso de que aún puedas terminarlo. |
Sorry, something went wrong.
|
Estaría bueno revisar si podemos etiquetar gente con el stale bot, supongo que después |
Sorry, something went wrong.
Reviewers
+1 more reviewer
5 participants
Descripción
Esta es la versión probada de la Github Action para ejecutar pospell con un comentario.
Uso
Los pasos del workflow son los siguientes:
pospell-fixpowrapypospellel bot encuentra cambios, se generará un commit directamente a tu PR.Referencias