filipedeschamps / meu-primeiro-jogo-multiplayer

Um jogo multiplayer bastante simples para testarmos o Socket.io
MIT License
1.25k stars 623 forks source link

Colidindo players, novos skins, som #32

Closed doriclaudino closed 4 years ago

doriclaudino commented 4 years ago

filipedeschamps commented 4 years ago

WOWWW!!!! 😍

Sensacionaaaal!!! Testei aqui no Gitpod e funcionou perfeito, muito bom!!!!!!!

Um ponto importante do seu PR: ele está movendo e alterando arquivos do experimento pontuacao-e-skin

Por exemplo, movendo o playground/pontuacao-e-skin/public/keyboard-listener.js para o seu experimento playground/player-collision-novos-pots/public/keyboard-listener.js

Para conseguir fazer o merge, você precisa devolver as alterações do outro experimento 👍

doriclaudino commented 4 years ago

Beleza! Vou ajustar.

doriclaudino commented 4 years ago

@filipedeschamps ajustei, criei um readme tmb, talvez alguém queira melhorar algo, tem muita coisa pra melhorar 🗡 💀

filipedeschamps commented 4 years ago

Fala meu caro! Desculpa a demora em retornar, correria do canal :) estamos quase lá para fazer o merge desse PR, só falta você desfazer essas alterações que ainda movem arquivos de um experimento para o outro, olha só:

image

image

image

image

image

E esses que removeram o conteúdo:

image

O jeito mais fácil de consertar:

Minha sugestão é: mova ou faça uma cópia da pasta do seu experimento para fora do repositório, algo realmente seguro. Depois disso, entre na master e resete o repositório git fetch && git reset --hard origin/master ... depois crie uma nova branch e mova aquela pasta do seu experimento para dentro.

Atenção: Caso esteja usando o Gitpod, tome cuidado para não resetar o ambiente inteiro e perder as alterações.

filipedeschamps commented 4 years ago

Fala meu caro! Com a abertura de uma outra issue eu vi que esse PR tinha sido reaberto, sensacional!! Fazendo o merge agora 👍 matou a pau!!!