Skip to content

Conversation

krzychb
Copy link
Contributor

@krzychb krzychb commented Mar 10, 2016

Ref: #1702
Another RP as previous failed I am not clear why.
Good luck 😄

krzychb and others added 6 commits March 8, 2016 20:45
two images renamed - trubleshooting
Menu entry "Upload Using:" has been removed
This is to make it's contents visible when browsing ota_updates
directory
...for all defined boards, i.e.:
1. Generic ESP8266 Module
2. ESPDuino (ESP-13 Module)
3. Adafruit HUZZAH ESP8266
4. ESPresso Lite 1.0
5. ESPresso Lite 2.0
6. NodeMCU 0.9 (ESP-12 Module)
7. NodeMCU 1.0 (ESP-12E Module)
8. Olimex MOD-WIFI-ESP8266(-DEV)
9. SparkFun ESP8266 Thing
10. SparkFun ESP8266 Thing Dev
11. SweetPea ESP-210
12. WeMos D1 R2 & mini
13. WeMos D1(Retired)
14. ESPino (ESP-12 Module)
15. ThaiEasyElec's ESPino
16. WifInfo
@igrr
Copy link
Member

igrr commented Mar 10, 2016

Don't worry about failing CI checks. I'll take care of that.
By the way, if you rename a file you should also update the menu here: https://github.com/esp8266/Arduino/blob/master/doc/reference_items.yml#L11
This one is used for HTML docs generation.

@krzychb
Copy link
Contributor Author

krzychb commented Mar 10, 2016

Thanks @igrr!
I will issue supplementary PR to cover update for HTML docs.
Krzysztof

@igrr
Copy link
Member

igrr commented Mar 10, 2016

You can simply push more commits here without creating another PR.

@igrr
Copy link
Member

igrr commented Mar 10, 2016

Okay, i think CI build might have been broken by the changes to boards.txt.

Could you please try removing this line in build script?
https://github.com/esp8266/Arduino/blob/master/tools/build.py#L43

@krzychb
Copy link
Contributor Author

krzychb commented Mar 10, 2016

@igrr,
This is fun discovering all internals of platform package deployment 👍
Krzysztof

igrr added a commit that referenced this pull request Mar 10, 2016
Remove "Upload Using:" from the menu since it is now misleading
@igrr igrr merged commit 2a6e8cc into esp8266:master Mar 10, 2016
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.

2 participants