Skip to content
This repository has been archived by the owner on Oct 23, 2024. It is now read-only.

Correção do bug do oni #75

Merged
merged 10 commits into from
Apr 21, 2024
Merged

Correção do bug do oni #75

merged 10 commits into from
Apr 21, 2024

Conversation

Crono209ggg
Copy link

@Crono209ggg Crono209ggg commented Apr 13, 2024

Estava ocorrendo um erro no linter porque uma linha de código estava faltando, mas agora corrigi.

@github-actions github-actions bot added the No C# label Apr 13, 2024
@rbertoche
Copy link
Owner

vou resolver os conflitos e mergir

@rbertoche
Copy link
Owner

Valeu!

@rbertoche
Copy link
Owner

esperando os testes

@rbertoche
Copy link
Owner

github idiota
eu ja fiz merge como assim conflitos

@rbertoche
Copy link
Owner

ah ele nao faz rebase assim F
perai vou ver o q fazer

@rbertoche
Copy link
Owner

rbertoche commented Apr 21, 2024

ta dificil mergir porcausa do seu branch estar cheio de merge velho
nao vale a pena ficar reaproveitando branches super antigos assim
infelizmente o pull request acaba atrapalhando um pouco porque maquia um pouco esse tipo de problema
na proxima tenta pegar uma cópia do meu branch pirata e começar a partir daí ao inves de ficar usando esse branch de meses atrás q ja ta cheio de merge q não interessa a mim nem a vc

quando as suas paradas são mergidas geralmente é um bom momento pra vc arrumar o seu branch principal e verificar se faltou algo por ex ao inves de ficar tentando manter um branch de coisa q ja foi mergida pra sempre

criar um branch separado a partir do branch pirata exatamente do ponto do meu repo pra cada PR ajuda imensamente nisso pq ai vc simplesmente nao vai ter problema nenhum na hora de fazer pull do seu branch pirata se nao tiver alterações nele

ah e outra coisa
aconteça o que acontecer não clique nas opções de "sync" ou "merge" pelo github pq isso tem consequências não obvias e é uma caixa preta sobre o que realmente tá acontecendo e só vai atrapalhar sua vida, fora q nos casos q ta dessincronizado geralmente o github nao faz a coisa certa
a única exceção é o botão de rebase and merge de PR, esse só faz rebase se não tiver conflitos então é mais seguro que não vai fazer errado, mas isso só aparece nas PRs pros seus repos

esse tipo de problema é mt comum pq o github tende a esconder em botões com nome simples e "automáticos" e efeito complicado e que não mostram nenhum tipo de aviso, mensagem ou explicação coisas complicadas e que requerem atenção especial e que o git na linha de comando ou em GUIs vai te explicar claramente. Acaba que por isso são funcionalidades que só funcionam pra usuários avançados e mesmo assim as vezes atrapalham, mesmo a gente achando que sabe o que vai acontecer exatamente

se vc tiver receio de jogar os branch fora pode apenas fazer rename e deixar pra ver depois se tem algo q precise recuperar neles

@rbertoche
Copy link
Owner

rbertoche commented Apr 21, 2024

além disso, com um branch velho assim vc consertar um bug nao garante q ja ta resolvido de vez o problema, pq quando fizer o merge o bug pode voltar

@rbertoche
Copy link
Owner

já fiz rebase localmente, eu vou fazer um merge pra marcar como mergido aqui ao inves de fechado pq é chato ficar aparecendo fechado
depois nao estranha o push force, é pra nao preservar os commits antigos ja mergidos

@rbertoche rbertoche merged this pull request into rbertoche:pirata Apr 21, 2024
5 of 6 checks passed
@rbertoche
Copy link
Owner

valeu pelo fix!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants