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

Documentation - Update supported IDEs and general improvements #8606

Merged
merged 1 commit into from
Oct 4, 2023

Conversation

lucasssvaz
Copy link
Collaborator

@lucasssvaz lucasssvaz commented Sep 6, 2023

Description of Change

Update supported IDEs and add link to more information about OTA updates.

Related links

Closes #8101

@lucasssvaz lucasssvaz added the Type: Documentation Issue pertains to Documentation of Arduino ESP32 label Sep 6, 2023
@PBrunot
Copy link
Contributor

PBrunot commented Sep 6, 2023

Why are you removing PlatformIO documentation ? It's a great IDE for this project.

@lucasssvaz
Copy link
Collaborator Author

cc @VojtechBartoska

@VojtechBartoska VojtechBartoska changed the title Documentation - Remove PlatformIO and general improvements Documentation - Updating supported IDEs and general improvements Sep 25, 2023
@lucasssvaz lucasssvaz changed the title Documentation - Updating supported IDEs and general improvements Documentation - Update supported IDEs and general improvements Sep 25, 2023
@VojtechBartoska
Copy link
Contributor

Why are you removing PlatformIO documentation ? It's a great IDE for this project.

Hello @PBrunot,

we have no doubts about PlatformIO being a great IDE but it's not officially supported by Espressif Systems right now. We are not able to manage and make sure that PlatformIO is supporting our latest version of Arduino ESP32 Core so due to that, we are deleting it from our Documentation to not confuse our Users. For using Arduino ESP32 Core in PlatformIO IDE, please refer to PlatformIO documentation.

@torntrousers
Copy link
Contributor

How about instead keep the info but just add a comment saying its independent and not supported by Espressif?

@VojtechBartoska
Copy link
Contributor

VojtechBartoska commented Sep 27, 2023

How about instead keep the info but just add a comment saying its independent and not supported by Espressif?

Actually, that's not bad approach.

@lucasssvaz Please revert those changes and add please to the https://github.com/espressif/arduino-esp32/blob/master/docs/source/installing.rst#installing-using-platformio Note block with the sentence above PlatformIO is not officially supported.

You can inspire from https://docs.espressif.com/projects/esp-idf/en/latest/esp32/third-party-tools/platformio.html#what-is-platformio

@lucasssvaz lucasssvaz force-pushed the doc/platformio branch 3 times, most recently from 2aedcb9 to 263ed5c Compare September 27, 2023 16:49
Copy link
Member

@pedrominatel pedrominatel left a comment

Choose a reason for hiding this comment

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

Just one minor change.

docs/source/installing.rst Outdated Show resolved Hide resolved
Copy link
Member

@pedrominatel pedrominatel left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@VojtechBartoska VojtechBartoska left a comment

Choose a reason for hiding this comment

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

Thanks for update @lucasssvaz

@VojtechBartoska VojtechBartoska added Status: Pending Status: Pending Merge Pull Request is ready to be merged and removed Status: Pending labels Oct 2, 2023
@P-R-O-C-H-Y P-R-O-C-H-Y removed their request for review October 3, 2023 13:19
@me-no-dev me-no-dev merged commit c156b89 into espressif:master Oct 4, 2023
49 checks passed
@ivankravets
Copy link
Contributor

Oh, we were not aware that Espressif was considering removing PlatformIO from its official documentation. It's surprising given our long collaboration and the significant value that PlatformIO has added to Espressif's business.

In response, we've created a separate issue (platformio/platform-espressif32#1225) to inform the community that version 2.x is the latest supported version. We hope to maintain the support and use of PlatformIO, as many Espressif users prefer it over Arduino IDE.

@VojtechBartoska
Copy link
Contributor

VojtechBartoska commented Nov 10, 2023

Hi @ivankravets, PlatformIO is not removed from our documentation.

We just added a note that keeping the support of our ESP32 Core in PlatformIO is maintained by PlatformIO and not by us to to make it clear for Users that it is a separate project and we are not able to handle the compatibility between our Core and PIO.

So we did not change anything, just called the current state properly. We will be glad if PIO still keeps support of our latest version.

We are open to discuss this more and make it clear :)

@thijstriemstra
Copy link

you're doing a really bad job handling this @VojtechBartoska. Creating confusion, reverting things. not sure if you work at espressif but this is a bad look for all of your employees.

@VojtechBartoska
Copy link
Contributor

VojtechBartoska commented Nov 10, 2023

Hello @thijstriemstra,

first of all, yes I do work in Espressif and I am Project Manager of Arduino Core Project. I hope not all contributors and users of this repository(project) thinks I am doing a bad job, I can say to you I am trying to do my best.

I am sorry that this Pull Request (change) created confusion to you, it was not meant to do this but exact opposite.

In general in this project we are trying to be as transparent as possible and we are really focused on user experience and trying to make this framework user friendly.

I will explain to you again, even though it is already explained in comments above.

This project is own and managed by Espressif and has wide base of different Users which use different IDEs.

PlatformIO is one of them, it is separate project providing IDE which we do not own, we are not able to keep it compatible. We have PlatformIO IDE and Arduino.cc IDE mentioned in our documentation page of this project because we want to mention most common (from our limited point of view) IDEs to have complete getting started docs for end user. If any IDE owner/maintainer contribute to the IDEs page in our Docs we will be more than happy.

Regarding this particular PR, Users are still coming to our repository mentioning PlatformIO issues we are not able to solve, due to that we initially though we will delete PIO documentation from our docs because the content was outdated and was bringing even more confusion and we are not able to keep this section up-to-date as we used different IDEs. After reconsidering this and also good feedback from @torntrousers, we decided to keep this PIO outdated docs there and just add a note that PIO is not developed by us. You are right that this note can be written in different way so we will change it. I hope it is now clear.

To summarize this, we will update PIO note in our docs to not bring a confusion, we are open to your feedback.

@Jason2866
Copy link
Collaborator

At least communication with the Platformio CEO before doing mandatory changes in documentation about support would have been a good idea. I hope for the many Platformio users there will be now a communication to clarify everything and mention Platformio prominent in the documentation as great IDE choice.
Bigger projects simply wouldn't be possible without Platformio. ArduinoIDE is to limited.

@VojtechBartoska
Copy link
Contributor

VojtechBartoska commented Nov 10, 2023

@Jason2866 Yea, we should ping @ivankravets in this PR before merging, we just did not consider adding this note just in our docs page as that major change. Support of 2.X Arduino Core version is still there maintained by PIO Team.

In general, there was no intention to hide anything or even upset someone, as I mentioned we just want to clarify we are not responsible and also able to help with PIO related issues.

And again, this project is open-source, anyone can contribute and we are not always able to ping/let know all related projects maintainers. We will try to improve this, thanks for feedback. 👍

@thijstriemstra
Copy link

In general, there was no intention to hide anything or even upset someone

Same here, sorry for sounding harsh, and thanks for rectifying things.

@timonsku
Copy link

In my own experience it can be a very significant support burden to deal with PIO specific issues.
PIO is a useful tool for many but I think its absolutely a reasonable step to say you can't provide support from your side as PIO is not in your control to fix these issue that people encounter. Its more beneficial for everyone if those problems are directed to the PIO issue tracker directly imo.
If people can't replicate their issue in the Arduino IDE then the issue is elsewhere, making sure of that saves a lot of time in debugging root causes.

@peteDDD
Copy link

peteDDD commented Dec 10, 2023

No doubt the Espressif folks were biased toward the (toy) Arduino IDE by the incorporation of ESP32 chips in new Arduino boards. Money talks...

Espressif is apparently not considering the quality and productivity of the IDEs. Real developers use real, robust IDEs like Visual Code and Visual Code Studio. Arduino IDE is a toy in comparison. The value-added to VSC by the PlatformIO team has been immense and spans many frameworks and platforms. This allows developers who must support multiple platforms to have one common toolset across those platforms which translates to real productivity for the developer.

I've been programming microprocessors since the 1970's with the Intel 8080 and studied under the program manager of the Motorola (later MicroChip) 6800 family back then. I've seen many generations of development tools. The ESP-IDF toolset has a user interface about as primitive as the tools we used back in the 1970s. VSC with PlatformIO makes programming and testing ESP32-family-based projects a pleasure in comparison. Meanwhile the Arduino IDE is for "makers" playing around, not for real product development, testing, and deployment.

@Jason2866
Copy link
Collaborator

@peteDDD There is nothing to add.

@rftafas
Copy link
Collaborator

rftafas commented Dec 11, 2023

Hi all,
This has already far deviated from the purpose of a PR. In order to keep this repo for its intended objective (sharing code), as well as the PRs (discussion about the PR itself) we will lock this and we suggest moving the discussion to the proper page or another discussion forum.

@espressif espressif locked as off-topic and limited conversation to collaborators Dec 11, 2023
@lucasssvaz lucasssvaz deleted the doc/platformio branch March 19, 2024 13:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Status: Pending Merge Pull Request is ready to be merged Type: Documentation Issue pertains to Documentation of Arduino ESP32
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add documentation: what happens when an update gets aborted