Skip to content

Update volatile.adoc: Add explanation to examples #712

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

Merged
merged 3 commits into from
Sep 23, 2020

Conversation

Granjow
Copy link
Contributor

@Granjow Granjow commented Dec 1, 2019

This change also adds a visual separator between the two code blocks so it is clear that they do not belong together.

This change also adds a visual separator between the two code blocks so it is clear that they do not belong together.
Copy link
Collaborator

@per1234 per1234 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Granjow. Thanks so much for your pull request. I have made a few suggestions for how it might be improved. Please let me know what you (and any other interested parties) think.

@Granjow
Copy link
Contributor Author

Granjow commented Dec 2, 2019

Hi @per1234, thanks a lot for your review and sorry for the bad quality of the MR. I was too much in a rush. Will improve it, but it is going to take a few days :)

@Granjow
Copy link
Contributor Author

Granjow commented Dec 16, 2019

@per1234 Any comments on the updated docs?

@Granjow
Copy link
Contributor Author

Granjow commented Feb 7, 2020

@per1234 ping. Can this MR be merged? Any open issues?

@Granjow
Copy link
Contributor Author

Granjow commented Sep 18, 2020

Guys, this PR is more than 1.5 years old. Would really appreciate to get this forward a bit. @kengdahl could you help me on that maybe?
Thanks.

@kengdahl
Copy link
Member

Hey @Granjow! I will refer this to @cmaglie, since this is more his thing.

Copy link
Member

@cmaglie cmaglie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested some changes in the loop.
For the rest the changes looks good.

@cmaglie cmaglie merged commit de455ae into arduino:master Sep 23, 2020
@Granjow
Copy link
Contributor Author

Granjow commented Sep 25, 2020

Thanks for merging!

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.

4 participants