search

pokemongovet / pokemongovet.github.io

Projeto para incentivar adoção de animais pessoas que amam pokemons virtuais
https://pokemongovet.github.io/
Other
102 stars 38 forks source link

Revert do commit que identava com tab/ update badge Fork / Correção: São Luís no Readme #5

Closed welksonramos closed 8 years ago

welksonramos commented 8 years ago
rcgivisiez commented 8 years ago

@welksonramos estou vendo sua PR, mas como tem muita alteração será discutida com mais membros.

fititnt commented 8 years ago

Re-identar milhares de linhas não é pequena alteração, e acho que concorda com isso.

No projeto, o padrão de identação usado é "dois espaços", conforme https://google.github.io/styleguide/htmlcssguide.xml?showone=Indentation#Indentation.

Se você puder não apenas reverter, mas remover o commit ainde você re-identa tudo com tab em vez de dois espaços, eu sugiro que ele seja aceito. Nesse caso é importante não apenas "re-identar de novo" mas implodir esse commit ali, senão a gente perde histórico anterior.

Depois que terminar esse PR, é interessante também deixar um título mas descritivo.

Fiz clone do repositório dele e, fora identação, por mim todo o resto estaria ok pra aceitação imediata.

welksonramos commented 8 years ago

Realizei o revert do commit que identava com tab, teve conflitos mas deu pra resolver. Sintam-se à vontade caso o PR esteja pronto hehe :smile:

fititnt commented 8 years ago

@welksonramos Não basta revert, tem que implodir o commit do histórico. Senão vai poluir o histórico.

No caso de repositórios que outras pessoas alteraram, é legal deixar o passado de alterações de outras pessoas.

Se não conseguir fazer o rebase pra remover (não apenas reverter) a identação, é mais fácil você criar um novo pull request e alterar apenas as linhas que precisa. É isso que eu pessoalmente faria.

fititnt commented 8 years ago

@welksonramos por motivos que expliquei aqui https://github.com/pokemongovet/pokemongo.vet.br/issues/6 ajudei a @icoffani e a @rcgivisiez pra fazer alterações de nome.

Com isso, esse PR foi fechado automaticamente. Eu vou ficar em contato direto com você pra resolver isso caso você não queira abrir a mesma alteração com essas sugestões de código

Suas alterações SÃO pertinentes. Eu não tenho como reabrir esse PR sem o master existir

welksonramos commented 8 years ago

@fititnt então se fizer o rebase e refazer as alterações resolve? Desculpe a pergunta, pois ainda sou leigo no git :)

fititnt commented 8 years ago

@welksonramos qual seu usuário do hangout? me diz ele aqui. O meu é fititnt at gmail.com. Eu faço uma call hoje e te ajudo passo a passo nisso.