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

AP_ExternalAHRS: Reserve Aeron ID #25777

Merged
merged 1 commit into from
Jan 20, 2024
Merged

Conversation

Jackwooh
Copy link
Contributor

We are planning to add our IMU support as an ExternalAHRS in the Ardupilot.

Copy link
Member

@IamPete1 IamPete1 left a comment

Choose a reason for hiding this comment

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

Per #25773 (comment) commit message needs fixing. AP_ExternalAHRS: Reserve aaeron ID

You should be able to do a interactive rebase and reword it. https://ardupilot.org/dev/docs/git-interactive-rebase.html

@Ryanf55 or myself could do it for you if you can't work it out.

@IamPete1
Copy link
Member

I have fixed up the commit message.

@Ryanf55
Copy link
Collaborator

Ryanf55 commented Dec 18, 2023

Per #25773 (comment) commit message needs fixing. AP_ExternalAHRS: Reserve aaeron ID

You should be able to do a interactive rebase and reword it. https://ardupilot.org/dev/docs/git-interactive-rebase.html

@Ryanf55 or myself could do it for you if you can't work it out.

Thanks for catching that, sorry, I wrongly assumed the title inherited the new commit message.

@Jackwooh
Copy link
Contributor Author

Jackwooh commented Jan 19, 2024

@tridge @IamPete1 Kindly update on the PR

@tridge tridge merged commit f67d895 into ArduPilot:master Jan 20, 2024
92 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants