Feb 13, 2022 · Oct 3, 2022 · Oct 3, 2022 · Oct 3, 2022 · Oct 3, 2022 · Oct 4, 2022
diff --git a/.github/workflows/pospell-on-demand.yml b/.github/workflows/pospell-on-demand.yml
 name: pospell
 on:
  issue_comment:
    types:
      - created

 jobs:
  fix-spell:
    name: Fix spelling
    runs-on: ubuntu-latest
    if: |
      github.event.comment.body == 'run-powrap'
      && github.event.issue.pull_request
    steps:
      - name: Confirmar ejecucion
        uses: actions/github-script@v6
        with:
          script: |
            const {owner, repo} = context.issue
            github.reactions.createForIssueComment({
              owner,
              repo,
              comment_id: context.payload.comment.id,
              content: "+1",
            });
      - uses: actions/checkout@v3
      - name: Preparar Python v3.10
        uses: actions/setup-python@v4
        with:
          python-version: "3.10"
          cache: "pip"
      - name: Instalar dependencias
        run: |
          sudo apt-get update
          sudo apt-get install -y hunspell hunspell-es gettext
          python -m pip install -r requirements.txt
      - name: Ejecutar Powrap
        run: powrap --quiet **/*.po
      - name: Revisar con Pospell
        run: python scripts/check_spell.py
        continue-on-error: true
      - name: Commit & Push changes
        uses: actions-js/push@master
        with:
          message: "auto: spell check"
          github_token: ${{ secrets.GITHUB_TOKEN }}
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,46 @@
		name: pospell
erickisos marked this conversation as resolved. OutdatedShow resolvedHide resolved
		on:
		issue_comment:
		types:
		- created

		jobs:
		fix-spell:
		name: Fix spelling
		runs-on: ubuntu-latest
		if: \|
		github.event.comment.body == 'run-powrap'
		&& github.event.issue.pull_request
		steps:
		- name: Confirmar ejecucion
		uses: actions/github-script@v6
		with:
		script: \|
		const {owner, repo} = context.issue
		github.reactions.createForIssueComment({
		owner,
		repo,
		comment_id: context.payload.comment.id,
		content: "+1",
		});
		- uses: actions/checkout@v3
		- name: Preparar Python v3.10
		uses: actions/setup-python@v4
		with:
		python-version: "3.10"
		cache: "pip"
		- name: Instalar dependencias
		run: \|
		sudo apt-get update
		sudo apt-get install -y hunspell hunspell-es gettext
		python -m pip install -r requirements.txt
		- name: Ejecutar Powrap
		run: powrap --quiet */.po
		- name: Revisar con Pospell
Copy link Collaborator rtobarOct 5, 2022 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Como mencionaba en el otro PR, pospell no hace ningún cambio local en tus archivos, sólo chequea si la ortografía es correcta o no, y retorna con status 0 o distinto a 0 dependiendo de si encontró errores. Esto significa que hacer chequeo ortográfico no tiene incidencia alguna en el paso siguiente (hacer un commit con los cambios locales), por lo que encuentro que no tiene mucho sentido ejecutar pospell en primer lugar -- aparte que hace que la acción se demore más en terminar. En mi opinión la acción debería estar centrada solo en ejecutar powrap y hacer el commit con los cambios que este encuentre, nada más (al menos esa era mi intención al crear#1786). Por eso también insistía en que los nombres del archivo, de la acción, etc, estuvieran todos centrados en powrap, no en pospell. erickisos reacted with heart emoji Copy link CollaboratorAuthor erickisosOct 5, 2022 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Oh vaya, got it, déjame hacer esos cambios de una, la verdad no me había quedado claro 😅
		run: python scripts/check_spell.py
		continue-on-error: true
erickisos marked this conversation as resolved. OutdatedShow resolvedHide resolved erickisos marked this conversation as resolved. OutdatedShow resolvedHide resolved
		- name: Commit & Push changes
rtobar marked this conversation as resolved. Show resolvedHide resolved
		uses: actions-js/push@master
Copy link Member ezio-melottiOct 5, 2022 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Tambien hayesta action que parece hacer el mismo, pero en el marketplace tiene 1k star (`actions-js/push` solo tiene 38). Nunca las he probado, así que no puedo decirte cuál es la mejor. Copy link CollaboratorAuthor erickisosOct 5, 2022 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Hice el cambio solo para hacer la prueba, si vemos que no nos funciona as expected, le damos de vuelta a la que teníamos ezio-melotti reacted with thumbs up emoji
		with:
		message: "auto: spell check"
		github_token: ${{ secrets.GITHUB_TOKEN }}
erickisos marked this conversation as resolved. Show resolvedHide resolved