-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[Mbed] Add to flake8 in workflow and fix python files (part #25193) #25249
Conversation
PR #25249: Size comparison from 61cf6f2 to 64ff74b Increases (6 builds for cc13x2_26x2, cc32xx, nrfconnect, telink)
Decreases (7 builds for bl702, cc13x2_26x2, telink)
Full report (33 builds for bl602, bl702, cc13x2_26x2, cc32xx, efr32, k32w, linux, nrfconnect, qpg, telink)
|
PR #25249: Size comparison from 31da720 to fb279a2 Decreases (1 build for cc32xx)
Full report (3 builds for cc32xx, linux)
|
PR #25249: Size comparison from b89e83b to 473306e Increases (1 build for cc32xx)
Full report (1 build for cc32xx)
|
473306e
to
0185b35
Compare
PR #25249: Size comparison from eff5773 to 0185b35 Decreases (1 build for cc32xx)
Full report (1 build for cc32xx)
|
0185b35
to
e6f7d7c
Compare
PR #25249: Size comparison from 536a0d9 to e6f7d7c Increases (1 build for cc32xx)
Full report (1 build for cc32xx)
|
e6f7d7c
to
a781ff3
Compare
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
ping @DamMicSzm |
a781ff3
to
3d6e3eb
Compare
PR #25249: Size comparison from da148bc to 3d6e3eb Increases (3 builds for bl702, esp32, nrfconnect)
Decreases (7 builds for bl602, cc32xx, esp32, k32w, psoc6, qpg)
Full report (45 builds for bl602, bl702, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg)
|
3d6e3eb
to
c409339
Compare
PR #25249: Size comparison from c8b49b9 to c409339 Increases (12 builds for bl602, bl702, nrfconnect, psoc6, qpg, telink)
Decreases (6 builds for k32w, telink)
Full report (61 builds for bl602, bl702, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
c409339
to
71e2cec
Compare
PR #25249: Size comparison from f467d6f to 71e2cec Increases (11 builds for esp32, psoc6, telink)
Decreases (6 builds for bl602, cc32xx, esp32, nrfconnect, telink)
Full report (61 builds for bl602, bl702, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
This pull request is a part of #25193 ( part fix python files in Mbed )
Problem
Python files need prevent things like syntax errors, typos, bad style, etc... it saves time for reviewing your code. Many python files needed bug fixes.
Changes
Fix all python files in mbed platform where linter find problem. Adding mbed platform to check with flake8.
Testing
CI will test and maybe some another manual testing