Skip to content

Conversation

@danilolmc
Copy link
Contributor

No description provided.

@javascript-translate-bot javascript-translate-bot requested a review from a team December 20, 2023 15:34
@danilolmc danilolmc changed the title updates 'symbol' article Symbol type Dec 22, 2023
@danilolmc danilolmc closed this Dec 22, 2023
@danilolmc danilolmc reopened this Dec 22, 2023
@nazarepiedady nazarepiedady merged commit 57f53a2 into javascript-tutorial:master Dec 22, 2023
@jonnathan-ls
Copy link
Contributor

jonnathan-ls commented Jan 13, 2024

@nazarepiedady

Lembre-se de registrar a revisão antes de aceitar (merge) o PR, além disso, também é preciso a revisão e aprovação de 02 revisores para estar alinhado com o processo que este repositório tem definido.

Há um BOT configurado que realiza a atualização de status do PR e da lista de tarefas em andamento, ele realiza isso com base nas interações do PR e no processo de adicionar uma revisão e consequentemente aprovação, adicionando flags como review needed, needs +1, ready to merge, etc..., além de marcar a tarefa no PR #1, como concluída.

Como ocorreu um merge direto sem passar pelos procedimentos, este PR ainda está com flag review needed, por mais que esteja fechado e o BOT de alguma forma deixou de atualizar a lista como concluído.

image

@nazarepiedady
Copy link
Member

nazarepiedady commented Jan 13, 2024

@jonnathan-ls, não estou nos meus melhores dias 😅, me esqueci dos protocolos de aprovação.

Não cometerei o mesmo erro três vezes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants