-
Notifications
You must be signed in to change notification settings - Fork 607
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
1 changed file
with
22 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
name: "Tag PRs with merge conflicts" | ||
on: | ||
# So that PRs touching the same files as the push are updated | ||
push: | ||
# So that the `dirtyLabel` is removed if conflicts are resolve | ||
# We recommend `pull_request_target` so that github secrets are available. | ||
# In `pull_request` we wouldn't be able to change labels of fork PRs | ||
pull_request_target: | ||
types: [synchronize] | ||
|
||
jobs: | ||
main: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: check if prs are dirty | ||
uses: eps1lon/actions-label-merge-conflict@releases/2.x | ||
with: | ||
dirtyLabel: "PR: tiene conflictos" | ||
removeOnDirtyLabel: "PR: lista para revisar" | ||
repoToken: "${{ secrets.GITHUB_TOKEN }}" | ||
commentOnDirty: "⚠️ Esta Pull Request tiene conflictos. Por favor, resuelvelos antes de que podamos evaluar los cambios." | ||
commentOnClean: "✅ ¡Los conflictos han sido resuletos! Un colaborador revisará pronto la Pull Request." |
92b1aad
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hola @midudev,
He notado que recientemente se ha integrado una actualización que afecta a la GitHub Action "Tag PRs with merge conflicts", y quería señalar un par de problemas que podrían necesitar tu atención:
Problema de Permisos al Etiquetar PRs: He observado un problema con la acción mencionada, donde muestra un error al intentar añadir o remover etiquetas en PRs debido a restricciones de permisos, específicamente
HttpError: Resource not accessible by integration
. Este problema impide que la acción funcione como se espera para PRs que tienen conflictos de merge.Actualización a Node.js 20: Según el anuncio oficial de GitHub, las GitHub Actions que utilizan Node.js 16 están quedando obsoletas, y se recomienda actualizar a Node.js 20 para mantener la compatibilidad y seguridad. Parece que la acción
eps1lon/actions-label-merge-conflict@releases/2.x
aún está utilizando Node.js 16, lo que podría conducir a advertencias o fallos en un futuro cercano.Una posible solución para el primer problema podría ser revisar la configuración de permisos del
GITHUB_TOKEN
o considerar el uso de un Personal Access Token (PAT) con los permisos adecuados para la gestión de etiquetas.Gracias!