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.

Sign up for GitHub

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

Jump to bottom

west.yml: hal_stm32: remove autoconf.h includes #73991

Merged
merged 1 commit into from
Jul 2, 2024
Merged

west.yml: hal_stm32: remove autoconf.h includes #73991

merged 1 commit into from
Jul 2, 2024

Conversation

erwango
Copy link
Member

@erwango erwango commented Jun 10, 2024

Sync with hal_stm32 PR

@erwango erwango requested a review from ycsin June 10, 2024 09:33
@zephyrbot zephyrbot added the size: XS A PR changing only a single line of code label Jun 10, 2024
@zephyrbot
Copy link
Collaborator

zephyrbot commented Jun 10, 2024 •
edited
Loading

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
hal_stm32 zephyrproject-rtos/hal_stm32@d12e21e zephyrproject-rtos/hal_stm32@2d9f676 (main) zephyrproject-rtos/[email protected]

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@zephyrbot zephyrbot added manifest manifest-hal_stm32 DNM This PR should not be merged (Do Not Merge) labels Jun 10, 2024
@zephyrbot zephyrbot removed the DNM This PR should not be merged (Do Not Merge) label Jul 1, 2024
henrikbrixandersen
henrikbrixandersen requested changes Jul 1, 2024
View reviewed changes
Copy link
Member

@henrikbrixandersen henrikbrixandersen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The commit massage should be updated to match the actual change.

@erwango
Copy link
Member Author

erwango commented Jul 1, 2024

The commit massage should be updated to match the actual change.

Thanks, fixed, ptal.

gmarull
gmarull previously approved these changes Jul 1, 2024
View reviewed changes
@str4t0m str4t0m changed the title west.yml: hal_stm32: prefix autoconf.h with zephyr/ west.yml: hal_stm32: remove autoconf.h includes Jul 1, 2024
str4t0m
str4t0m previously approved these changes Jul 1, 2024
View reviewed changes
ycsin
ycsin previously approved these changes Jul 1, 2024
View reviewed changes
@erwango
west.yml: hal_stm32: Remove autoconf.h
0c227da 
Sync with hal_stm32 PR

Signed-off-by: Erwan Gouriou <[email protected]>
@erwango erwango requested review from gmarull and str4t0m July 2, 2024 09:27
@erwango erwango added this to the v3.7.0 milestone Jul 2, 2024
@aescolar aescolar merged commit 6700a15 into zephyrproject-rtos:main Jul 2, 2024
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@henrikbrixandersen henrikbrixandersen henrikbrixandersen approved these changes

@ycsin ycsin ycsin approved these changes

@aescolar aescolar aescolar approved these changes

@gmarull gmarull Awaiting requested review from gmarull

@str4t0m str4t0m Awaiting requested review from str4t0m

Assignees

@erwango erwango

Labels
manifest manifest-hal_stm32 size: XS A PR changing only a single line of code
Projects
None yet
Milestone
v3.7.0
Development

Successfully merging this pull request may close these issues.
7 participants
@erwango @zephyrbot @henrikbrixandersen @ycsin @gmarull @aescolar @str4t0m