Skip to content

Conversation

@chiomaso
Copy link

Changed from byte to integer for analog read

Changed from byte to integer for analog read
@ToniCorinne
Copy link

Thanks for following our tutorial! I'm going to close this out to leave the repo usable for others, but let us know if you have any questions!

@chiomaso
Copy link
Author

chiomaso commented Dec 21, 2016 via email

@ToniCorinne
Copy link

Hi @chiomaso - you didn't do anything incorrectly on this. What I meant when I said I was closing out your pull request to leave the repo functional for others is simply that if I accepted your pull request, it would fix the error in the code that our tutorial is based around. If the error is no longer there, the repo would not be a useful teaching tool for others following along with the tutorial.

However, in a regular GitHub project (not a demo repo like this), I would have absolutely accepted your pull request, which would also automatically close this out, but only after making the changes to the code that you were offering. I hope that clarifies!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants