-
Notifications
You must be signed in to change notification settings - Fork 8
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
Document datami sdk update #57
Document datami sdk update #57
Conversation
.travis.yml
Outdated
@@ -6,8 +6,6 @@ android: | |||
licenses: | |||
- android-sdk-license-.+ | |||
before_install: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Essas alterações são necessárias para o funcionamento dessa release ? Caso não, sugira retirá-las desse PR e criar um focado somente nelas.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sim, são necessárias para que PR externos ao repositório passem no Travis. Há um PR somente para o Travis em que essas modificações são realizadas: Try to fix travis error
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alterar os pontos comentados
4730827
to
da494ff
Compare
Removi as modificações solicitadas |
Boa! Mas não entendi, a mudança agora é só em uma linha do README ? Subiu algo errado ? |
Essa issue era pra ser apenas documentando o processo de atualização do SDK do Datami, então sim, apenas no README mesmo porém acabei removendo a parte em que isso é documentado sem querer, já arrumo hehe |
Ah sim haha, blz Me dá um @ quando tiver finalizado |
da494ff
to
613b40b
Compare
Acabei removendo a parte que descrevia o processo de atualização junto com as modificações extras ao PR hehe |
Foi documentado como é feito o processo de inclusão do SDK do Datami no projeto. A documentação foi adicionada a README.md com o intuito de auxiliar quem for realizar essa atualização que deve ser feita de maneira constante.
Resolve #52