Stack should be merged when merge commit is already pushed
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 58/100
Direção de pesquisa
Rastreie o entry point existente gh stack sync e a lógica que determina se os PRs empilhados foram mesclados. Compare o tratamento da base com o cenário de merge commit enviado manualmente descrito aqui. Considera-se concluído quando todos os PRs da stack são reconhecidos como mesclados e fechados quando a base da stack contém suas alterações.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Reproduce step
Let's say you have this stacked PRs.
main <- PR#1 <- PR#2 <- PR#3
Now, you manually create merge commit and push it to the main.
prv_main <- PR#1 <- PR#2 <- PR#3
\---------------------+-------new_main
Current Behavior
PR#1is closed, because it's base ismain, and GitHub automatically closes already merged PRs.PR#2andPR#3are NOT closed, because their bases aren'tmain(they point to previous PR's tip), so they still have diff remaining, and PR isn't closed.
Expected Behavior
- All PRs should be closed.
How to fix this
Stacked PR should check it's mergeability against the whole stack's base (e.g. main in this case).
Basically the whole point of stacked PR is that "your PR is pointing to another PR, but they are treated as pointing to base".
Why this should be fixed
- These PRs are shown as "open PRs" while they are already merged.
- This will help 3rd-party integrations -- e.g. bors manually creates octopus merge commit of PRs and push it to main.
I know that you can manually gh stack sync, but I think this should be done automatically.
- Linguagem predominante
- Go
- Estrelas
- 1.5k
- Forks
- 73
- Merge médio
- 1d 8h
- PRs com merge (30d)
- 7
Guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de github/gh-stack
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 92/100
-
feature request topic: cli - general
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
-
feature request topic: auto-merge
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
-
bug topic: docs
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 68/100
Todas as issues de github/gh-stack
Issues semelhantes
-
bug github_actions
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
registrystack/registry-stack#1393 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
JakeChampion/lang#10213 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
oasisprotocol/oasis-sdk#2523 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100