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

Renomeação dos nomes das funções de leitura dos pinos #53

Closed
wants to merge 4 commits into from
Closed

Renomeação dos nomes das funções de leitura dos pinos #53

wants to merge 4 commits into from

Conversation

vinebarbosa
Copy link
Contributor

Resolvida issue #52

@OtacilioN
Copy link
Owner

Opa @ViniciosSilva811 acho super válida a contribuição, e de fato acabam sendo nomes melhores do que os já existentes.

Me preocupo apenas com a compatibilidade para programas escritos anteirormente em brasilino, que deixariam de funcionar uma vez que estes comandos sejam renomeados.

Afim de garantir que os programs antigos ainda funcionem, e proporcionar uma experiência melhor para novos programas com o ajuste no nome, recomendo que seja feita uma adição do comando, em vez de uma substituição, mantendo ainda o nome antigo, mas deixando claro em toda a documentação e exemplos que o comando atual é a nomeclatura nova.

@OtacilioN
Copy link
Owner

@ViniciosSilva811 uma outra dica, muito provavelmente teu git config --global user.email não tá refletindo teu email do GitHub, então na aba de contribuidores da biblioteca teu nome não está aparecendo pois ele não consegue associar a tua conta.

@vinebarbosa
Copy link
Contributor Author

@ViniciosSilva811 uma outra dica, muito provavelmente teu git config --global user.email não tá refletindo teu email do GitHub, então na aba de contribuidores da biblioteca teu nome não está aparecendo pois ele não consegue associar a tua conta.

E como eu faço para concertar esse problema? (Sou novato com git). Uso git config --global user.email "meu email"?

@vinebarbosa
Copy link
Contributor Author

vinebarbosa commented Oct 13, 2019

Opa @ViniciosSilva811 acho super válida a contribuição, e de fato acabam sendo nomes melhores do que os já existentes.

Me preocupo apenas com a compatibilidade para programas escritos anteirormente em brasilino, que deixariam de funcionar uma vez que estes comandos sejam renomeados.

Afim de garantir que os programs antigos ainda funcionem, e proporcionar uma experiência melhor para novos programas com o ajuste no nome, recomendo que seja feita uma adição do comando, em vez de uma substituição, mantendo ainda o nome antigo, mas deixando claro em toda a documentação e exemplos que o comando atual é a nomeclatura nova.

Concordo plenamente. Logo logo eu farei um commit com devidas as modificações.

@OtacilioN
Copy link
Owner

Opa @ViniciosSilva811 acho super válida a contribuição, e de fato acabam sendo nomes melhores do que os já existentes.
Me preocupo apenas com a compatibilidade para programas escritos anteirormente em brasilino, que deixariam de funcionar uma vez que estes comandos sejam renomeados.
Afim de garantir que os programs antigos ainda funcionem, e proporcionar uma experiência melhor para novos programas com o ajuste no nome, recomendo que seja feita uma adição do comando, em vez de uma substituição, mantendo ainda o nome antigo, mas deixando claro em toda a documentação e exemplos que o comando atual é a nomeclatura nova.

Concordo plenamente. Logo logo eu farei um commit com devidas as modificações.

Okay @ViniciosSilva811, no aguardo das modificações solicidadas pra a gente poder adicionar oficialmente a biblioteca.

Copy link
Owner

@OtacilioN OtacilioN left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Não remover comandos antigos para manter compatibilidade com códigos já escritos em Brasilino, apenas adicionar essa opção nova para que as pessoas possam usá-la no local de lerDigital.

@@ -76,8 +76,8 @@
#define escreverAnalogico(pino, valor) analogWrite(pino, valor)
#define escreverDigital(pino, valor) digitalWrite(pino, valor)

#define lerDigital(pino) digitalRead(pino)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Não remover comandos antigos para manter compatibilidade com códigos já escritos em Brasilino, apenas adicionar essa opção nova para que as pessoas possam usá-la no local de lerDigital.

@vinebarbosa
Copy link
Contributor Author

Fiz uma nova pull request atendendo as modificações solicitadas.

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.

2 participants