Skip to content
This repository has been archived by the owner on Jan 25, 2024. It is now read-only.

Add minimal board definition for Arduino Uno #293

Merged
merged 2 commits into from
Jun 6, 2019
Merged

Conversation

trickeydan
Copy link
Contributor

@trickeydan trickeydan commented May 18, 2019

This is useful as the parent class for #66 and #65

Missing from this board definition:

  • PWM on Digital Pins
  • I2C
  • SPI

These are not in scope for this. This PR only implements what is necessary for #65 and #66. Issues should be added to implement this support in the future.

@trickeydan trickeydan added this to the v0.5.0 milestone May 18, 2019
@trickeydan trickeydan added the enhancement New feature or request label May 18, 2019
@trickeydan trickeydan marked this pull request as ready for review May 26, 2019 17:54
j5/boards/arduino/uno.py Outdated Show resolved Hide resolved
docs/conf.py Outdated Show resolved Hide resolved
j5/boards/arduino/uno.py Outdated Show resolved Hide resolved
@trickeydan
Copy link
Contributor Author

I've done a rebase to clean up the history here, but it should be good to go now.

@trickeydan trickeydan requested review from kierdavis and sedders123 and removed request for kierdavis June 6, 2019 12:50
Fix misordered arguments in cast.

Looks like I was quite tired when I wrote that originally..

Remove setup_arduino function
@codeclimate
Copy link

codeclimate bot commented Jun 6, 2019

Code Climate has analyzed commit 58f7aaf and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (95% is the threshold).

This pull request will bring the total coverage in the repository to 98.5% (0.0% change).

View more on Code Climate.

@trickeydan trickeydan merged commit ea33b47 into master Jun 6, 2019
@delete-merged-branch delete-merged-branch bot deleted the arduino-uno branch June 6, 2019 14:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants