Skip to content

Conversation

fxdmhtt
Copy link

@fxdmhtt fxdmhtt commented Jul 11, 2025

Summary

Sync the following PRs:
apache/nuttx-apps#3020
apache/nuttx-apps#2980
apache/nuttx-apps#2967
apache/nuttx-apps#2957
apache/nuttx-apps#2955
apache/nuttx-apps#2944
apache/nuttx-apps#2487

Impact

none

Testing

It has been tested on apache/nuttx-apps. I have also tested it on open-vela.

@CLAassistant
Copy link

CLAassistant commented Jul 11, 2025

CLA assistant check
All committers have signed the CLA.

@xiaoxiang781216
Copy link
Collaborator

need keep the individual patch as it instead merging into one.

@fxdmhtt
Copy link
Author

fxdmhtt commented Jul 11, 2025

I don't quite understand why we need to keep a individual patch? These 7 PRs are actually repeated improvements on these files, and they should essentially be 7 commits under one PR.
Is there a simple way to migrate these 7 PRs?

@xiaoxiang781216
Copy link
Collaborator

I don't quite understand why we need to keep a individual patch? These 7 PRs are actually repeated improvements on these files, and they should essentially be 7 commits under one PR. Is there a simple way to migrate these 7 PRs?

I mean you should keep the original patch as it instead merging into two patch:
apache/nuttx-apps#3020
apache/nuttx-apps#2980
apache/nuttx-apps#2967
apache/nuttx-apps#2957
apache/nuttx-apps#2955
apache/nuttx-apps#2944
apache/nuttx-apps#2487

Copy link

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the Stale label Aug 24, 2025
@github-actions github-actions bot closed this Sep 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants