SerenityOS/serenity

PR Labeler: Does not properly relablel PRs

Aberta

#17.271 aberto em 2 de fev. de 2023

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (0 responsável)C++ (3.328 forks)batch import
help wantedtask list

Métricas do repositório

Stars
 (33.171 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

We added the handy PR labeler CI action, but it doesn't seem to add labels properly in the following cases:

  • When a review is submitted by someone other than the author, change the state from "needs review" to "waiting for author feedback"
  • When a review is approved, move the state to "community approved" or "maintainer approved" based on who did the approval.
  • When a PR is detected to have conflicts, add the "has conflicts" label
  • When changes are pushed, remove the "waiting for author feedback" label and add "needs review"

Guia do colaborador

Direção de pesquisa
Examine o arquivo YAML do fluxo de trabalho do rotulador de PR existente (provavelmente em .github/workflows) e identifique por que ele falha ao reetiquetar PRs em submissão de revisão, aprovação, detecção de conflitos e eventos de push. Em seguida, adicione os gatilhos de evento necessários e a lógica condicional para lidar com cada caso.
Pilha de tecnologia
github actions
Domain
ci cd
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitGitHub Actions
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70