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

[Examples] WiFi and Ethernet examples organization #5830

Conversation

pedrominatel
Copy link
Member

@pedrominatel pedrominatel commented Nov 2, 2021

Summary

This PR is to change the WiFi and Ethernet examples organization in a separated folder. Two folders were created for WiFi and Ethernet examples.

Impact

WiFi examples now are in a different folder location. This may cause some impacts on external references to the examples.

@pedrominatel pedrominatel self-assigned this Nov 2, 2021
@pedrominatel
Copy link
Member Author

PTAL @me-no-dev and evaluate the risks and pros and cons of this change.

@me-no-dev
Copy link
Member

There are no risks other than maybe breaking some links somewhere on the web.

@@ -1,4 +1,4 @@
name=WiFi
name=WiFi and Ethernet
Copy link
Contributor

Choose a reason for hiding this comment

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

This breaks dependencies, do not change the name.

@me-no-dev
Copy link
Member

@pedrominatel why not also move ETH.h and ETH.cpp into the new lib? (you would need to adjust CMakeLists.txt as well)

@pedrominatel
Copy link
Member Author

PTAL @me-no-dev

@me-no-dev me-no-dev merged commit a2d7c0d into espressif:master Nov 4, 2021
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