-
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
Add controller tests to the build. #10729
Add controller tests to the build. #10729
Conversation
/rebase |
d547822
to
06e2995
Compare
/rebase |
06e2995
to
5fe4edb
Compare
/rebase |
5fe4edb
to
e99fc4d
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. |
@mrjerryjohns Sorry, can you take a look at the merge conflict, and build failures? |
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. |
A number of important controller-side tests got added recently, but they are not being run in CI. Well, let's add them.
e99fc4d
to
b58e83f
Compare
PR #10729: Size comparison from ea10a5d to cde94a9 Increases above 0.2%:
Increases (34 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Decreases (3 builds for mbed, nrfconnect, p6)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
c7a89c7
to
348d960
Compare
PR #10729: Size comparison from 507ba15 to 348d960 Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
* Add controller tests to the build. A number of important controller-side tests got added recently, but they are not being run in CI. Well, let's add them. * Fixed up the builds * Fixed up the tests to pass
A number of important controller-side tests got added recently, but they
are not being run in CI. Well, let's add them.