-
Notifications
You must be signed in to change notification settings - Fork 3k
adr: Recommend CMSIS component source structure #13464
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
Conversation
@hugueskamba, thank you for your changes. |
ca7f3f4
to
fa9f23a
Compare
01fad41
to
fce9b2b
Compare
This PR cannot be merged due to conflicts. Please rebase to resolve them. |
Jenkins CI Test : ✔️ SUCCESSBuild Number: 5 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
5c6eb97
to
d7856c4
Compare
Jenkins CI Test : ❌ FAILEDBuild Number: 11 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
9b4ae20
to
3bded3d
Compare
The following force-pushes:
|
3bded3d
to
5840064
Compare
This force-push amends the adr for the change above. |
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 good, thanks!
Jenkins CI Test : ✔️ SUCCESSBuild Number: 12 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
See docs/adr/0001_cmsis_component_source_structure_recommendations.md For details information.
5840064
to
834e530
Compare
Pull request has been modified.
This force-push amends the |
Jenkins CI Test : ✔️ SUCCESSBuild Number: 13 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
Summary of changes
Added an ADR to recommend component source structure.
The PR has two commits. One adds the adr and the other one implements the changes proposed.
The PR depends on:
#13469
Impact of changes
Migration actions required
Documentation
Pull request type
Test results
Reviewers