-
-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
docs,esp8266,esp32: Replace deprecated network interface constants. #16155
Conversation
3dcd3c2
to
242e78b
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
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. |
There was a problem hiding this 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.
58b6634
to
44d4003
Compare
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>
44d4003
to
5dfbd43
Compare
Summary
As noted in #16119 (comment) the deprecated
network.STA_IF
andnetwork.AP_IF
constants were still listed in a number of places instead of the recommendednetwork.WLAN.IF_STA
andnetwork.WLAN_IF_AP
constants.This PR updates all remaining instances.
This PR has a sibling in micropython-lib: micropython/micropython-lib#933
Testing