-
Notifications
You must be signed in to change notification settings - Fork 11
feat: mtu exchange interface update #686
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
Open
XavierBernhardt-Philips
wants to merge
33
commits into
main
Choose a base branch
from
feature/split_AttMtuInterface
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
✅
|
| Descriptor | Linter | Files | Fixed | Errors | Warnings | Elapsed time |
|---|---|---|---|---|---|---|
| ✅ ACTION | actionlint | 6 | 0 | 0 | 0.36s | |
| ✅ CPP | clang-format | 15 | 2 | 0 | 0 | 1.69s |
| ✅ CPP | cppcheck | 15 | 0 | 0 | 0.82s | |
| ✅ DOCKERFILE | hadolint | 1 | 0 | 0 | 0.39s | |
| ✅ JSON | jsonlint | 7 | 0 | 0 | 0.14s | |
| ✅ JSON | prettier | 7 | 2 | 0 | 0 | 1.46s |
| markdownlint | 6 | 1 | 10 | 0 | 2.32s | |
| ✅ MARKDOWN | markdown-table-formatter | 6 | 1 | 0 | 0 | 1.19s |
| ✅ REPOSITORY | checkov | yes | no | no | 27.09s | |
| ✅ REPOSITORY | git_diff | yes | no | no | 0.79s | |
| ✅ REPOSITORY | grype | yes | no | no | 39.32s | |
| ✅ REPOSITORY | ls-lint | yes | no | no | 0.06s | |
| ✅ REPOSITORY | secretlint | yes | no | no | 43.03s | |
| ✅ REPOSITORY | syft | yes | no | no | 1.44s | |
| ✅ REPOSITORY | trivy | yes | no | no | 7.97s | |
| ✅ REPOSITORY | trivy-sbom | yes | no | no | 1.16s | |
| ✅ REPOSITORY | trufflehog | yes | no | no | 9.11s | |
| lychee | 33 | 1 | 0 | 74.11s | ||
| ✅ YAML | prettier | 9 | 1 | 0 | 0 | 1.43s |
| ✅ YAML | v8r | 9 | 0 | 0 | 5.3s | |
| ✅ YAML | yamllint | 9 | 0 | 0 | 0.45s |
Detailed Issues
⚠️ SPELL / lychee - 1 error
[IGNORED] tcp://localhost:1234/target/echo/programmer | Unsupported: Error creating request client: builder error for url (tcp://localhost:1234/target/echo/programmer)
[ERROR] http://mcd.rou.st.com/modules.php?name=mcu | Network error: error sending request for url (http://mcd.rou.st.com/modules.php?name=mcu) Maybe a certificate error?
[TIMEOUT] https://st.com/ | Timeout
📝 Summary
---------------------
🔍 Total..........238
✅ Successful.....235
⏳ Timeouts.........1
🔀 Redirected.......0
👻 Excluded.........0
❓ Unknown..........0
🚫 Errors...........1
Errors in README.md
[TIMEOUT] https://st.com/ | Timeout
Errors in .github/ISSUE_TEMPLATE/add-or-update-hal-driver.md
[ERROR] http://mcd.rou.st.com/modules.php?name=mcu | Network error: error sending request for url (http://mcd.rou.st.com/modules.php?name=mcu) Maybe a certificate error?
⚠️ MARKDOWN / markdownlint - 10 errors
.github/ISSUE_TEMPLATE/add-or-update-hal-driver.md:10 MD025/single-title/single-h1 Multiple top-level headings in the same document [Context: "How to import a HAL driver"]
CHANGELOG.md:83 MD024/no-duplicate-heading Multiple headings with the same content [Context: "⚠ BREAKING CHANGES"]
CHANGELOG.md:87 MD024/no-duplicate-heading Multiple headings with the same content [Context: "Features"]
CHANGELOG.md:94 MD024/no-duplicate-heading Multiple headings with the same content [Context: "Bug Fixes"]
CHANGELOG.md:103 MD024/no-duplicate-heading Multiple headings with the same content [Context: "Features"]
CHANGELOG.md:122 MD024/no-duplicate-heading Multiple headings with the same content [Context: "Bug Fixes"]
CHANGELOG.md:140 MD024/no-duplicate-heading Multiple headings with the same content [Context: "⚠ BREAKING CHANGES"]
CHANGELOG.md:144 MD024/no-duplicate-heading Multiple headings with the same content [Context: "Features"]
CHANGELOG.md:151 MD024/no-duplicate-heading Multiple headings with the same content [Context: "Features"]
CHANGELOG.md:162 MD024/no-duplicate-heading Multiple headings with the same content [Context: "Bug Fixes"]
See detailed reports in MegaLinter artifacts
Your project could benefit from a custom flavor, which would allow you to run only the linters you need, and thus improve runtime performances. (Skip this info by defining FLAVOR_SUGGESTIONS: false)
- Documentation: Custom Flavors
- Command:
npx [email protected] --custom-flavor-setup --custom-flavor-linters ACTION_ACTIONLINT,CPP_CPPCHECK,CPP_CLANG_FORMAT,DOCKERFILE_HADOLINT,JSON_JSONLINT,JSON_PRETTIER,MARKDOWN_MARKDOWNLINT,MARKDOWN_MARKDOWN_TABLE_FORMATTER,REPOSITORY_CHECKOV,REPOSITORY_GIT_DIFF,REPOSITORY_GRYPE,REPOSITORY_LS_LINT,REPOSITORY_SECRETLINT,REPOSITORY_SYFT,REPOSITORY_TRIVY,REPOSITORY_TRIVY_SBOM,REPOSITORY_TRUFFLEHOG,SPELL_LYCHEE,YAML_PRETTIER,YAML_YAMLLINT,YAML_V8R
…lientSt and GattServerSt
e76b70a to
e647760
Compare
…nterface # Conflicts: # CMakeLists.txt
daantimmer
approved these changes
Dec 17, 2025
CMakeLists.txt
Outdated
| emil | ||
| GIT_REPOSITORY https://github.com/philips-software/amp-embedded-infra-lib.git | ||
| GIT_TAG d73a5b1b107f3f46ad99317dd0c226c5f22060a2 # unreleased | ||
| GIT_TAG 1ed62fa1a8d461a207c27b2adf694b7c7977de40 # unreleased |
Contributor
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.
update once emil is merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.

No description provided.