Skip to content

docs,esp8266,esp32: Replace deprecated network interface constants. #16155

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

Merged
merged 4 commits into from
Nov 8, 2024

Conversation

projectgus
Copy link
Contributor

@projectgus projectgus commented Nov 5, 2024

Summary

As noted in #16119 (comment) the deprecated network.STA_IF and network.AP_IF constants were still listed in a number of places instead of the recommended network.WLAN.IF_STA and network.WLAN_IF_AP constants.

This PR updates all remaining instances.

This PR has a sibling in micropython-lib: micropython/micropython-lib#933

Testing

  • I haven't tested these changes on device, but I believe the new constants exist on all ports with Wi-Fi and these changes were 95% done by regex so there should be minimal chance of breakage.

Copy link

codecov bot commented Nov 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.57%. Comparing base (df6b40a) to head (5dfbd43).
Report is 4 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #16155   +/-   ##
=======================================
  Coverage   98.57%   98.57%           
=======================================
  Files         164      164           
  Lines       21345    21345           
=======================================
  Hits        21041    21041           
  Misses        304      304           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@dpgeorge dpgeorge left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

There are a few commit messages with two signed-off lines that you may want to fix.

@projectgus projectgus force-pushed the doc/network_wlan_constants branch 2 times, most recently from 58b6634 to 44d4003 Compare November 5, 2024 06:03
Removes the deprecated network.[AP|STA]_IF form from the docs.

This work was funded through GitHub Sponsors.

Signed-off-by: Angus Gratton <angus@redyak.com.au>
Removes the deprecated network.[AP|STA]_IF form from the esp32 port

This work was funded through GitHub Sponsors.

Signed-off-by: Angus Gratton <angus@redyak.com.au>
Removes the deprecated network.[AP|STA]_IF form from the esp8266 port

This work was funded through GitHub Sponsors.

Signed-off-by: Angus Gratton <angus@redyak.com.au>
Removes the deprecated network.[AP|STA]_IF form from unit tests.

This work was funded through GitHub Sponsors.

Signed-off-by: Angus Gratton <angus@redyak.com.au>
@dpgeorge dpgeorge force-pushed the doc/network_wlan_constants branch from 44d4003 to 5dfbd43 Compare November 8, 2024 01:04
@dpgeorge dpgeorge merged commit 5dfbd43 into micropython:master Nov 8, 2024
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants