loopinfinito / loopinfinito.com.br

Site do Loop Infinito
loopinfinito.com.br
68 stars 15 forks source link

Revisão do post 'Ponto e vírgula em JS' #262

Closed yurigoytacaz closed 10 years ago

almirfilho commented 10 years ago

Não estou entendendo porque tem commits antigos do caio nesse pull request. Sabe dizer o que está havendo @caiogondim?

almirfilho commented 10 years ago

Já saquei qual o problema. @yurigoytacaz fez um pull request para o master, mas não podemos fazer merge no master ainda. Portanto, vou fechar esse pull request e pedir para que o @yurigoytacaz submeta um novo pull request para a branch do post post-js-ponto-e-virgula. Pode ser @yurigoytacaz?

almirfilho commented 10 years ago

@yurigoytacaz espera só o @caiogondim responder sobre a parada do . Dai você já muda logo (ou não).

yurigoytacaz commented 10 years ago

Claro! Sem problemas. Vou providenciar.

2013/10/9 Almir Filho notifications@github.com

Já saquei qual o problema. @yurigoytacaz https://github.com/yurigoytacazfez um pull request para o master, mas não podemos fazer merge no master ainda. Portanto, vou fechar esse pull request e pedir para que o @yurigoytacaz https://github.com/yurigoytacaz submeta um novo pull request para a branch do post post-js-ponto-e-virgula. Pode ser @yurigoytacaz https://github.com/yurigoytacaz?

— Reply to this email directly or view it on GitHubhttps://github.com/loopinfinito/loopinfinito.com.br/pull/262#issuecomment-25998378 .

yurigoytacaz commented 10 years ago

É só ele mudar quando ele for dar merge, po. Não?

2013/10/9 Almir Filho notifications@github.com

@yurigoytacaz https://github.com/yurigoytacaz espera só o @caiogondimhttps://github.com/caiogondimresponder sobre a parada do . Dai você já muda logo (ou não).

— Reply to this email directly or view it on GitHubhttps://github.com/loopinfinito/loopinfinito.com.br/pull/262#issuecomment-25998453 .

almirfilho commented 10 years ago

@yurigoytacaz dá pra selecionar as linhas na hora do merge? Falei isso pra garantir que a gente não esqueça.