-
Notifications
You must be signed in to change notification settings - Fork 119
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
Support esp32 boards #2038
Support esp32 boards #2038
Conversation
// Catch error produced by arduino-cli, but actually it's not an error: | ||
// When User added a new URL into `extra.txt` file it causes that | ||
// arduino-cli tries to read new JSON but it's not existing yet | ||
// so it fails with error "no such file or directory" | ||
// To avoid this and make a good UX, we'll force call `updateIndexes` | ||
return updateIndexesInternal(wsPath, cli).then(() => listBoards(cli)); | ||
return updateIndexesInternal(wsPath, cli).then(() => |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The comment above is stale now. Needs update.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤔
It still actual, but I'll edit it a bit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍🏻
2c39f79
to
a25e12b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
This PR includes:
extra.txt
: updated arduino-cli returns "ENOENT" code instead of6
when JSON file is not found and ourcatch
does not work as expected. This bug does not break a loading new file, but it shows an error message, and list of boards appears only after clicking "Update" button or opening of the upload popup next time.xod-dev/servo/servo-device
andxod/gpio/pwm-write
to use the bundled library and work fine on ESP32