-
Notifications
You must be signed in to change notification settings - Fork 28
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
Conversation
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. |
@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"? |
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. |
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.
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) |
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.
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.
Fiz uma nova pull request atendendo as modificações solicitadas. |
Resolvida issue #52