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

Update sbf.h #147

Merged
merged 1 commit into from
Feb 7, 2024
Merged

Update sbf.h #147

merged 1 commit into from
Feb 7, 2024

Conversation

SeptentrioGNSS
Copy link
Contributor

only overwrite Stream1 in the autoconfig of the Septentrio Receiver, al other streams are untouched

only overwrite Stream1 in the autoconfig of the Septentrio Receiver, al other streams are untouched
@bkueng bkueng merged commit 3393191 into PX4:main Feb 7, 2024
@SeptentrioGNSS
Copy link
Contributor Author

@bkueng maybe a stupid question but this is now merged into the main, with the next official release of PX4, will it be included? As there is currently v1.14 released but anything from this point foreward like 1.14.x or 1.15 wil this include this merge?

@bkueng
Copy link
Member

bkueng commented Feb 8, 2024

It will be included in the next major/minor release, but not in point releases like 1.14.x.

@SeptentrioGNSS
Copy link
Contributor Author

What is the procedure to get this merged for the next point release? Is this a direct ask to certain developers?

@bkueng
Copy link
Member

bkueng commented Feb 15, 2024

I quickly did it as it's a small change, but we typically only backport important bug fixes.
PX4/PX4-Autopilot#22759

@SeptentrioGNSS
Copy link
Contributor Author

Ok thank you for this. For the future what is the normal process of getting thing like this and bigger chunks of code included in the next possible release? Is it to create a PR to the release 1.14 branch instead of the main?

@bkueng
Copy link
Member

bkueng commented Feb 16, 2024

Is it to create a PR to the release 1.14 branch instead of the main?

Yes exactly

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