Estabelece restrições para trabalhos da Arduino Compile para placas sem interface Serial nativa #73
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
*compatibilidade para placas sem comunicação Serial (Ex.: Arduino Gemma, Franzininho) *Cria uma variável para as placas que diz se a placa possui comunicação serial *condição para executar jobs (trabalhos) caso a placa não tenha comn Serial habilitado
Se refere as placas que possuem comunicação serial por meio de interface USB padrão do Arduino, que é a mesma serial do Monitor Serial. Essa variável é necessária, pois há placas (arduino-gemma e micronucleus-franzininho) que trabalham com processador que não suporta o comando `iniciarSerial()` dos exemplos.
Volta a testar com Arduino Compile programas que estavam descontinuados em #68
Retira a restrição de compile para interface serial, visto que este código não usa interface Serial.
|
@ErickSimoes e @OtacilioN, corrigi os bugs com a versão da arduino-cli. Agora limitei a Action Arduino Compile para a versão arduino/[email protected]; visto que, a partir da próxima versão são inseridos mudanças que quebram o estado do código atual. Como a arduino-cli não possui uma versão 1.0, pretende-se manter esta por hora: garantir estabilidade e funcionamento. A PR está pronta para Review e Merge! |
OtacilioN
approved these changes
Apr 4, 2021
There was a problem hiding this comment.
Perfeito para Merge, @ErickSimoes pode revisar e apertar o botão ✅
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Sumário
Esta Pull Request visa estabelecer restrições de trabalhos (jobs) para placas sem interface Serial nativa (Ex.: Arduino Gemma e Franzininho).
Outras Informações
Além disso, insere os códigos
controleGradual.inoeASCII.inopara compile, por meio da Action Arduino Compile.EDIT (03/04/2021):
Limita a versão da arduino-cli para a versão 0.17.0, pois versões superiores apresentaram mudanças com quebra.
Testes
Foram realizados testes em minha fork, disponível em SteffanoP/Brasilino.
Nas Actions deste projeto é possível visualizar alguns testes.
Como também nas Actions de minha fork para ver todos os testes.
Descrição das mudanças
analogicoSerial.inoquarentaedois.inoLuminosidade.inoTemperatura.inoUltrassom-HC-SR04.inocontroleGradual.inoASCII.inoBenefícios
Estabelece compatibilidade da Integração Contínua Arduino Compile com placas de interface serial não nativa, ou seja, as placas
arduino-gemmaemicronucleus-franzininhoirão ser restringidos em etapas (steps) do trabalho (job), resolvendo os problemas constatados em #68. A restrição do compile se vale apenas para exemplos em que a interface Serial não é nativa (exemplos com comandos comoiniciarSerial()eescreverSerial()passam a não ser válidos de teste para essas placas).Possíveis desvantagens
Ainda há a necessidade de adicionar um script para realizar o compile de forma automática, atualmente é necessário alterar manualmente o destino de compile no
./.github/workflows/compile.ymlsempre que for adicionado um exemplo.Problemas Relacionados
Resolves #68