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: gap and gatt refactored/improved due to embedded-infra changes #136

Merged
merged 24 commits into from
Jul 12, 2023

Conversation

gabrielsantosphilips
Copy link
Contributor

@gabrielsantosphilips gabrielsantosphilips commented May 24, 2023

  • Redundant code removed.
  • embedded-infra-lib hash updated to the latest version.
  • Central tracing class improved.
  • Error handling on connection failure improved.
  • Add function to exchange MTU on GAP central.
  • Add function to set phy to 2 Mpbs.

@github-actions
Copy link
Contributor

github-actions bot commented May 24, 2023

🦙 MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Elapsed time
✅ ACTION actionlint 5 0 0.05s
⚠️ MARKDOWN markdownlint 20 14 2 2.17s
⚠️ MARKDOWN markdown-link-check 20 18 35.07s
✅ MARKDOWN markdown-table-formatter 20 14 0 0.78s
✅ REPOSITORY checkov yes no 15.86s
✅ REPOSITORY dustilock yes no 0.03s
✅ REPOSITORY gitleaks yes no 72.62s
✅ REPOSITORY git_diff yes no 0.74s
✅ REPOSITORY kics yes no 18.84s
✅ REPOSITORY secretlint yes no 22.92s
✅ REPOSITORY syft yes no 0.57s
✅ REPOSITORY trivy yes no 3.62s
✅ XML xmllint 17 0 0 0.59s
✅ YAML prettier 7 1 0 1.25s
✅ YAML v8r 7 0 5.74s
✅ YAML yamllint 7 0 0.55s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

hal_st/middlewares/ble_middleware/GapCentralSt.cpp Outdated Show resolved Hide resolved
hal_st/middlewares/ble_middleware/GapSt.hpp Outdated Show resolved Hide resolved
hal_st/middlewares/ble_middleware/GattClientSt.cpp Outdated Show resolved Hide resolved
hal_st/middlewares/ble_middleware/GattClientSt.cpp Outdated Show resolved Hide resolved
hal_st/middlewares/ble_middleware/GattServerSt.cpp Outdated Show resolved Hide resolved
hal_st/middlewares/ble_middleware/GattServerSt.cpp Outdated Show resolved Hide resolved
hal_st/middlewares/ble_middleware/GattServerSt.cpp Outdated Show resolved Hide resolved
hal_st/middlewares/ble_middleware/GapCentralSt.cpp Outdated Show resolved Hide resolved
hal_st/middlewares/ble_middleware/GapCentralSt.cpp Outdated Show resolved Hide resolved
hal_st/middlewares/ble_middleware/GapSt.cpp Outdated Show resolved Hide resolved
hal_st/middlewares/ble_middleware/GapSt.hpp Outdated Show resolved Hide resolved
hal_st/middlewares/ble_middleware/GapSt.hpp Outdated Show resolved Hide resolved
gabrielsantosphilips and others added 2 commits June 26, 2023 11:55
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@gabrielsantosphilips gabrielsantosphilips changed the title feat: gap and gatt refactored due to embedded-infra changes chore: gap and gatt refactored due to embedded-infra changes Jun 27, 2023
@gabrielsantosphilips gabrielsantosphilips changed the title chore: gap and gatt refactored due to embedded-infra changes chore: gap and gatt refactored/improved due to embedded-infra changes Jul 7, 2023
@gabrielsantosphilips gabrielsantosphilips marked this pull request as ready for review July 7, 2023 10:45
@gabrielsantosphilips gabrielsantosphilips requested a review from a team as a code owner July 7, 2023 10:45
@gabrielsantosphilips gabrielsantosphilips changed the title chore: gap and gatt refactored/improved due to embedded-infra changes feat: gap and gatt refactored/improved due to embedded-infra changes Jul 10, 2023
@gabrielsantosphilips gabrielsantosphilips enabled auto-merge (squash) July 12, 2023 12:34
@gabrielsantosphilips gabrielsantosphilips merged commit e6c276b into main Jul 12, 2023
13 checks passed
@gabrielsantosphilips gabrielsantosphilips deleted the feature/gap_gatt_st_refactor branch July 12, 2023 12:34
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