Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invert push priority and Carry with up and down arrows #258

Closed

Conversation

guilhermercaetano
Copy link
Contributor

@guilhermercaetano guilhermercaetano commented Jun 28, 2016

Não dê PR por enquanto. Ainda precisa definir como que o herói vai fazer para puxar o objeto, visto que até agora ele só empurra na direção do movimento. #253 e #252

O commit ficou um pouco grande pq o editor que eu uso substituiu todos os caracteres de TAB por 4 espaços, o que ele fez dessa fez por alguma razão. Isso fez o commit perder um pouco da objetividade, do que eu realmente mudei. O arquivo HeroControl.cs dá pra ver as mudanças, mas do Hero.cs, o que eu mudei foi só a ordem de ativação na função DoAction (o que era if() passou para else if (), e vice versa).

@ednaldomoreira
Copy link
Contributor

Guilherme eu fi\z o pull request #255 que inverte a prioridade

@ednaldomoreira
Copy link
Contributor

@guilhermercaetano pq vc não trabalha com mais de uma branch ?
Veja esse post do slack que eu fiz
https://muvucagames.slack.com/files/ednaldomoreira/F1L11BUNM/branchs.png

@ednaldomoreira
Copy link
Contributor

@StefanWerW estas alterações excessivas (usar espaço em vez de tab) nos arquivos podem causar conflitos nos pull requests ?

@StefanWerW
Copy link
Member

Nao sei dizer. Mas acho que sim. Se o IDE de alguém tiver configurado pra alterar essa formatação, ele ira mudar todo o arquivo quando quiser mudar simplesmente uma parte, e assim podendo criar conflito com alguém que modificou outra parte do mesmo arquivo.

@guilhermercaetano
Copy link
Contributor Author

guilhermercaetano commented Jun 28, 2016

@ednaldomoreira então, podemos aprovar então seu PR? #255
Nesse PR eu tentei criar uma branch e mandar o PR por ela. Esse pull_box eu criei só pra essa tarefa.
Eu reverti a mudança nos Hero.cs, assim vamos evitar qualquer possibilidade de conflito.

@otaconwow
Copy link
Member

o PR #255 foi merged @guilhermercaetano

@StefanWerW
Copy link
Member

@guilhermercaetano
Copy link
Contributor Author

Agora já que o #255 foi aprovado, a gente pode dar sequencia nesse aqui.

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

Successfully merging this pull request may close these issues.

None yet

5 participants