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

Add machine states for run-time testing. #412

Closed
wants to merge 6 commits into from

Conversation

t0mpr1c3
Copy link
Contributor

@t0mpr1c3 t0mpr1c3 commented Aug 6, 2020

Referencing #413.

The communication methods were already implemented in the API, but there were no corresponding states in the finite state machine. This PR tries to address that.

Subsumes changes from API v5 to API v6 in previous PR (#411).

@t0mpr1c3 t0mpr1c3 changed the title Add communication methods for run-time testing. Add machine states for run-time testing. Aug 6, 2020
@t0mpr1c3 t0mpr1c3 force-pushed the 1.0.0-test branch 3 times, most recently from b3ff08e to 865e727 Compare August 6, 2020 20:10
@dl1com
Copy link
Contributor

dl1com commented Aug 6, 2020

Due to the merge of the other PR, some conflicts arised...since the PR is coming from a branch of your account I can't resolve them on my own :-/

@dl1com dl1com marked this pull request as draft August 6, 2020 20:19
@t0mpr1c3
Copy link
Contributor Author

t0mpr1c3 commented Aug 6, 2020

Due to the merge of the other PR, some conflicts arised...since the PR is coming from a branch of your account I can't resolve them on my own :-/

I fixed the issue. Anyway it's OK to mark it as a draft for the time being. I will be changing things with the API here and there as I fix up the firmware.

@t0mpr1c3 t0mpr1c3 force-pushed the 1.0.0-test branch 6 times, most recently from 595fa57 to 49a151e Compare August 6, 2020 22:25
@t0mpr1c3 t0mpr1c3 force-pushed the 1.0.0-test branch 3 times, most recently from eed87f1 to 4795534 Compare August 8, 2020 12:16
@t0mpr1c3 t0mpr1c3 force-pushed the 1.0.0-test branch 2 times, most recently from 9056297 to 3c46c93 Compare August 10, 2020 17:32
@t0mpr1c3 t0mpr1c3 force-pushed the 1.0.0-test branch 9 times, most recently from 7981cf0 to 009caaa Compare August 10, 2020 19:47
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