-
Notifications
You must be signed in to change notification settings - Fork 7.4k
tests: Bluetooth: Tester: VCP BSIM test #86664
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
base: main
Are you sure you want to change the base?
Conversation
3532311
to
b817f40
Compare
12dd064
to
44c4b1b
Compare
9e0b85c
to
1aa846b
Compare
1aa846b
to
a87222f
Compare
a87222f
to
d8f12f3
Compare
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.
Copilot reviewed 14 out of 26 changed files in this pull request and generated no comments.
Files not reviewed (12)
- tests/bluetooth/tester/Kconfig.sysbuild: Language not supported
- tests/bluetooth/tester/boards/nrf52_bsim.conf: Language not supported
- tests/bluetooth/tester/boards/nrf52_bsim.overlay: Language not supported
- tests/bluetooth/tester/boards/nrf5340bsim_nrf5340_cpuapp.conf: Language not supported
- tests/bluetooth/tester/boards/nrf5340bsim_nrf5340_cpuapp.overlay: Language not supported
- tests/bsim/bluetooth/compile.nrf5340bsim_nrf5340_cpuapp.sh: Language not supported
- tests/bsim/bluetooth/compile.sh: Language not supported
- tests/bsim/bluetooth/tester/CMakeLists.txt: Language not supported
- tests/bsim/bluetooth/tester/Kconfig: Language not supported
- tests/bsim/bluetooth/tester/boards/nrf52_bsim.overlay: Language not supported
- tests/bsim/bluetooth/tester/compile.sh: Language not supported
- tests/bsim/bluetooth/tester/prj.conf: Language not supported
e48ab9f
to
43accc8
Compare
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 like the change to this file does not apply when assigning reviewers and labels
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.
Correct. Only the maintainers file owner itself is notified.
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.
That's pretty interesting, because that means there's a chicken-and-egg situation here, right?
We cannot files to MAINTAINERS.yml before they exist, and the file isn't applied when added together :)
It's not a big issue, but was a bit surprising as generally changes to files in a PR is applied to CI
43accc8
to
c5a3ce4
Compare
dc4169f
to
e22626d
Compare
Adds BSIM testing of the VCP features of the BT Tester. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>
e22626d
to
6e122cf
Compare
|
Adds BSIM testing of the VCP features of the BT Tester.
Part of #86073