Skip to content

adafruit_esp32spi_socket.py:readline hardcoded end of line marker #94

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
graemorrison opened this issue Mar 25, 2020 · 4 comments
Closed
Labels
enhancement New feature or request

Comments

@graemorrison
Copy link

readline in adafruit_esp32spi_socket uses a hardcoded end of line marker as '\r\n', this obviously does not work for all systems, it should be made definable.

I have updated the routine to do this and tested it out on a pyportal but do not have permissions to push a new branch back to the repo, please see the attached updated code (used a .txt suffix as .py is not allowed)

adafruit_esp32spi_socket.py.txt

regards,
Grae Morrison

@FoamyGuy
Copy link
Contributor

FoamyGuy commented Mar 25, 2020

@graemorrison if you are up to it can you go ahead and make a Pull Request with these changes? There is a guide here on how to do this for circuitpython projects: https://learn.adafruit.com/contribute-to-circuitpython-with-git-and-github. You'll make a fork of the project and then make your changes within a branch of your fork and push it. Then come back to this repo and create the Pull Request selecting your fork/branch to get merged into the main one. If not no worries I can try to get it created later tonight.

@graemorrison
Copy link
Author

Thanks for the pointer @FoamyGuy - I'm more than happy to go down that road, will get on with it and let you know,

@kattni kattni added the enhancement New feature or request label Apr 29, 2020
@tekktrik
Copy link
Member

Resolved by PR #150!

@FoamyGuy
Copy link
Contributor

closed by #150

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants