-
Notifications
You must be signed in to change notification settings - Fork 27
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
[Para Revisão] - Capítulo 3 - Native - DeviceAccounts #61
base: master
Are you sure you want to change the base?
[Para Revisão] - Capítulo 3 - Native - DeviceAccounts #61
Conversation
@juarezpaf Segue PR para revisão |
@drdpedroso Cara, tá com conflito, creio que seja na inclusão do "*+3.42 DeviceAccounts *"... |
@gabrielclima Como devo proceder nesse caso? Simplesmente removo o "*+3.42 DeviceAccounts *"? |
Desculpe, na real o problema foi esse da imagem. Com as atualizações e inclusões de itens na documentação original, a galera atualizou os índices aqui... o que o ocorre é que o item 3.41 Vibration que está no seu índice agora virou 3.55 > Vibration. Na hora que você enviou, estava tudo ok. Mas antes de revisarem seu PR, acabaram revisando o PR do carinha que atualizou o índice ou um de nosso administradores atualizou os índice na mão para poder manter atualizado. Nesse momento, seu PR ficou inválido. O que se deve fazer nesse caso é o seguinte, você deve dar um pull para atualizar seu fork se baseando na branch A gente vai pensar numa melhor forma disso não acontecer. |
@gabrielclima Creio que agora ja de para mergear :) Obrigado pelas dicas |
Opa, e aí cara, beleza?! Por favor, pode retirar do seu PR os commits referente aos sumários, tanto do README.md geral como do capítulo. A partir de agora, não precisa mais linkar nos sumários. Faremos isso separadamente. Obrigado! 😄 |
@ednilsonamaral Feito :) |
No description provided.