-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Enable Closure linux app and TestScripts in CI. #39884
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
Enable Closure linux app and TestScripts in CI. #39884
Conversation
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.
Code Review
This pull request enables the closure
linux app in the CI by updating the necessary build and test scripts. One suggestion was provided to improve code readability.
PR #39884: Size comparison from ce90b61 to 457f718 Full report (54 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #39884: Size comparison from ce90b61 to 09859a7 Full report (52 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #39884: Size comparison from ce90b61 to 75e2cf8 Full report (71 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #39884: Size comparison from ce90b61 to 9d2e353 Increases above 0.2%:
Full report (71 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
* script changes for ci * Update scripts/build/builders/host.py * add closure to run test suites * Restyled by autopep8 * Fix CI build issues * Fix Add closure tests to CI * add closure app build * enable tracing support for linux * Adding remove handler for test events * Restyled by whitespace * Removing closure build for certification tests --------- Co-authored-by: Restyled.io <commits@restyled.io>
Summary
This PR aims to enable the closure linux app on CI , for testscripts under src/pythontesting can be executed acroos the app.
it also enalbes closure test script testing for closure linux app on CI.
Note: PR will increase CI time.
Testing
Should be successful run of CI.
Readability checklist
The checklist below will help the reviewer finish PR review in time and keep the
code readable:
descriptive
“When in Rome…”
rule (coding style)
See: Pull Request Guidelines