silveiravh
commented
7 years ago Não consegui fechar o antigo pull request e estou tentando atualizar o antigo. Professor, não sei se os arquivos foram atualizados no pull request. Por favor verifique,
2017-09-25 22:19 GMT-03:00 Vinícius Henrique vhsilveira2@gmail.com:
Professor, acabei de tentar um novo pull request com as alterações que fizemos. Favar verificar. Abs
2017-09-25 22:15 GMT-03:00 Flávio Coutinho notifications@github.com:
@fegemo requested changes on this pull request.
Pessoal, a maior parte das coisas que falei na revisão do pull request continua. Para aceitar o pull request do jeito que ele está, vou gastar umas 2h minhas pra ajustar o que está indevido e vocês podem perder muitos pontos por isso. Vocês pretendem alterar ou vão deixar como está?
In _gitignore https://github.com/fegemo/cefet-games-meow-au/pull/11#discussion_r140937117 :
@@ -0,0 +1,72 @@ +## Java
Este arquivo _gitignore continua aqui, mas por que vocês o adicionaram? O git usa um .gitignore e nós já temos um.
In _nb-gradle-properties.xml https://github.com/fegemo/cefet-games-meow-au/pull/11#discussion_r140937157 :
@@ -0,0 +1,3 @@ +
Idem. Este arquivo não deveria ter sido incluído no repositório.
In core/src/br/cefetmg/games/minigames/Bot.java https://github.com/fegemo/cefet-games-meow-au/pull/11#discussion_r140937289 :
@@ -0,0 +1,328 @@ +package br.cefetmg.games.minigames;
Conforme conversamos em sala, as classes específicas de cada minigame devem residir como classes internas dentro da classe do respectivo minigame.
In core/src/br/cefetmg/games/minigames/Colision.java https://github.com/fegemo/cefet-games-meow-au/pull/11#discussion_r140937327 :
@@ -0,0 +1,126 @@ +package br.cefetmg.games.minigames;
Idem.
In core/src/br/cefetmg/games/minigames/MiniGame.java https://github.com/fegemo/cefet-games-meow-au/pull/11#discussion_r140937361 :
@@ -29,7 +29,7 @@ protected float maxDuration; private float timeSpentOnInstructions; private float timeSpentPlaying;
- private MiniGameState state;
- MiniGameState state;
Aqui continua alterado, sem justificativa.
In core/src/br/cefetmg/games/minigames/Obstacle.java https://github.com/fegemo/cefet-games-meow-au/pull/11#discussion_r140937406 :
@@ -0,0 +1,37 @@ +package br.cefetmg.games.minigames;
Deveria estar dentro da classe de um minigame.
In core/src/br/cefetmg/games/screens/PlayingGamesScreen.java https://github.com/fegemo/cefet-games-meow-au/pull/11#discussion_r140937468 :
@@ -246,5 +248,11 @@ public void onGameResumed() { FINISHED_GAME_OVER, FINISHED_WON } +
- @Override
- public void resize(int width, int height) {
A pergunta não foi respondida e a alteração continua aqui...
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/fegemo/cefet-games-meow-au/pull/11#pullrequestreview-65078693, or mute the thread https://github.com/notifications/unsubscribe-auth/AQJ2OvC7wrtkq10ipZqOai3vJkQCFyWAks5smFAjgaJpZM4PaZsC .
-- Vinícius Henrique Silveira
Computer Engeneering Student at CEFET-MG
“Life begins at the end of your comfort zone.” — Neale Donald Walsch
-- Vinícius Henrique Silveira
Computer Engeneering Student at CEFET-MG
“Life begins at the end of your comfort zone.” — Neale Donald Walsch
fegemo
Professor, acabei de tentar um novo pull request com as alterações que fizemos. Favar verificar. Abs
2017-09-25 22:15 GMT-03:00 Flávio Coutinho notifications@github.com:
-- Vinícius Henrique Silveira
Computer Engeneering Student at CEFET-MG
“Life begins at the end of your comfort zone.” — Neale Donald Walsch