MOD-WIFI-ESP8266(-DEV) update by Stanimir-Petev · Pull Request #8297 · esp8266/Arduino · GitHub
Skip to content

MOD-WIFI-ESP8266(-DEV) update#8297

Merged
earlephilhower merged 6 commits into
esp8266:masterfrom
Stanimir-Petev:master
Oct 31, 2021
Merged

MOD-WIFI-ESP8266(-DEV) update#8297
earlephilhower merged 6 commits into
esp8266:masterfrom
Stanimir-Petev:master

Conversation

@Stanimir-Petev

Copy link
Copy Markdown
Contributor

Added Flash mode and reset method selection options for Olimex board MOD-WIFI-ESP8266(-DEV)

Added Flash mode and reset method selection options for Olimex board MOD-WIFI-ESP8266(-DEV)

@earlephilhower earlephilhower left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This isn't going to pass CI. You need to update tools/boards.txt.py and regenerate the boards.txt file from it,, not manually edit boards.txt.

Plus, it seems like you're doing both normal definitions and menu overridden ones which is not correct. See line 5832-5833 and 5849-5850.

@Stanimir-Petev

Stanimir-Petev commented Sep 3, 2021

Copy link
Copy Markdown
Contributor Author

@earlephilhower earlephilhower self-requested a review September 3, 2021 22:13

@earlephilhower earlephilhower left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, thanks!

@earlephilhower earlephilhower merged commit 3f5a76c into esp8266:master Oct 31, 2021
hasenradball pushed a commit to hasenradball/Arduino that referenced this pull request Nov 18, 2024
Added Flash mode and reset method selection options for Olimex board MOD-WIFI-ESP8266(-DEV)
The addition of the FlashMode and ResetMethod selection options via boards.txt.py generation file.
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