Skip to content
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

feat!: refactor ble middleware #346

Conversation

gabrielsantosphilips
Copy link
Contributor

@gabrielsantosphilips gabrielsantosphilips commented Jun 24, 2024

  • Includes and definitions were fixed to be used with STM32WBxx and STM32WBAxx.
  • Logic to handle Set Data Length changed: just update set data length if starts with unexpected values, otherwise starts to configure PHY.
  • BLE stack initialization moved from GAP ST to Transport Layer. GAP/GATT files only deal with ACI/HCI api.

Copy link
Contributor

github-actions bot commented Jun 24, 2024

🦙 MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Elapsed time
✅ ACTION actionlint 5 0 0.02s
✅ CPP clang-format 15 2 0 1.02s
✅ DOCKERFILE hadolint 1 0 0.1s
✅ JSON jsonlint 7 0 0.16s
✅ JSON prettier 7 2 0 1.43s
⚠️ MARKDOWN markdownlint 6 1 7 1.84s
⚠️ MARKDOWN markdown-link-check 6 2 52.13s
✅ MARKDOWN markdown-table-formatter 6 1 0 0.9s
✅ REPOSITORY checkov yes no 26.04s
✅ REPOSITORY git_diff yes no 0.45s
✅ REPOSITORY grype yes no 20.4s
✅ REPOSITORY secretlint yes no 44.55s
✅ REPOSITORY trivy yes no 5.07s
✅ REPOSITORY trivy-sbom yes no 3.6s
✅ REPOSITORY trufflehog yes no 9.46s
⚠️ SPELL lychee 29 3 58.07s
✅ YAML prettier 7 1 0 1.45s
✅ YAML v8r 7 0 5.57s
✅ YAML yamllint 7 0 0.22s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

@gabrielsantosphilips gabrielsantosphilips force-pushed the feature/prepare-ble-middleware-to-receive-new-microcontroller branch 2 times, most recently from 76f6fa3 to 4c32bb4 Compare June 24, 2024 12:05
@gabrielsantosphilips gabrielsantosphilips force-pushed the feature/prepare-ble-middleware-to-receive-new-microcontroller branch from 4c32bb4 to a0186fc Compare June 24, 2024 12:10
@gabrielsantosphilips gabrielsantosphilips changed the title chore: prepare ble middleware to receive new microcontroller feat!: prepare ble middleware to receive new microcontroller Jun 26, 2024
@gabrielsantosphilips gabrielsantosphilips marked this pull request as ready for review June 26, 2024 09:44
@gabrielsantosphilips gabrielsantosphilips requested a review from a team as a code owner June 26, 2024 09:44
@gabrielsantosphilips gabrielsantosphilips changed the title feat!: prepare ble middleware to receive new microcontroller feat!: refactor ble middleware Jun 26, 2024
@gabrielsantosphilips gabrielsantosphilips force-pushed the feature/prepare-ble-middleware-to-receive-new-microcontroller branch from 7a8ce6c to 26625cf Compare July 3, 2024 10:04
@gabrielsantosphilips gabrielsantosphilips force-pushed the feature/prepare-ble-middleware-to-receive-new-microcontroller branch from 26625cf to 732805c Compare July 3, 2024 10:34
@gabrielsantosphilips gabrielsantosphilips merged commit 6703195 into main Jul 3, 2024
23 checks passed
@gabrielsantosphilips gabrielsantosphilips deleted the feature/prepare-ble-middleware-to-receive-new-microcontroller branch July 3, 2024 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants