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

Isdale #71

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Isdale #71

wants to merge 2 commits into from

Conversation

MauiJerry
Copy link

I added a couple examples for Pair1 and Pub/Sub that are derived from what is in the ReadTheDocs. I also documented these a bit over on Hackaday.io.
Reference Issue #43 and #67

examples extended with Async and Pub/Sub
@codypiersall
Copy link
Owner

Thanks for the PR, @MauiJerry. Examples are always welcome!

I do request one change though, and that is to use the same naming conventions used elsewhere in the library--you can make the changes, or I can merge it and make the changes myself. It makes no difference to me!

The changes would be to rename file names and variables to use snake_case instead of camelCase.

Thanks for the PR!

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