Skip to content

Salocinx master #415

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

Closed
wants to merge 7 commits into from
Closed

Salocinx master #415

wants to merge 7 commits into from

Conversation

z3t0
Copy link
Collaborator

@z3t0 z3t0 commented Feb 7, 2017

#413

@salocinx

I needed to recreate the PR to add the changelog commit. I will give the code another look in the next few days and then merge within the week.

Thank you very much! Your contribution is greatly appreciated. If possible, could you summarize in a sentence or two any difficulties you may have had in creating this protocol spec? Just something to add to the wiki for any future coders that want to add their own protocols.

@z3t0
Copy link
Collaborator Author

z3t0 commented Feb 11, 2017

The branch has become out of date again, should be a trivial fix. Also the new protocol needs to be added to the relevant example sketches. And we need documentation to explain the protocol specs

@ArminJo ArminJo force-pushed the master branch 3 times, most recently from 308a1c6 to e8f8f2d Compare September 16, 2020 11:13
@ArminJo ArminJo force-pushed the master branch 3 times, most recently from c606b7e to 052e160 Compare October 12, 2020 09:11
@ArminJo
Copy link
Collaborator

ArminJo commented Oct 19, 2020

Please rebase the code if the protocol is still required.

@ArminJo ArminJo closed this Oct 19, 2020
@ArminJo ArminJo deleted the salocinx-master branch March 3, 2021 11:56
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.

3 participants