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

Added NodeMCU board support. #173

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

Added NodeMCU board support. #173

wants to merge 2 commits into from

Conversation

flsg
Copy link

@flsg flsg commented Jun 2, 2017

It adds the following features:

  • Added NodeMCU support

It fixes the following issues:

  • Fixed Wemos D1 R2 pin D6, D7
  • Changed input validation for analogwrite, value range 0-1023 for ESP8266 and 0-255 for the rest

@carlosperate
Copy link
Owner

Hi,

Thank you very much for the submission, it'd be great to add this to the Ardublockly repository, but this PR contain a lot of changes not related to the required changes (and a lot of formatting changes). It'd be great if you update this PR or create a new one with just the changes needed for this additional board support. It'd be even better if you could also separate the fixes to the Wemos support into a different PR, as that would make it easier for me to review and test.

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