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

Wiki build error fix: rangefinder labels #1894

Merged
merged 1 commit into from Oct 23, 2019
Merged

Wiki build error fix: rangefinder labels #1894

merged 1 commit into from Oct 23, 2019

Conversation

Hwurzburg
Copy link
Contributor

Eliminates rangefinder errors caused by moving to multiple rangefinders.....no content changes...

brunoolivieri
brunoolivieri approved these changes Aug 11, 2019
View reviewed changes
Copy link
Collaborator

@brunoolivieri brunoolivieri left a comment

Choose a reason for hiding this comment

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

LGTM

khancyr
khancyr approved these changes Aug 11, 2019
View reviewed changes
Copy link
Contributor

@khancyr khancyr left a comment

Choose a reason for hiding this comment

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

Seems good but I think that we should wait for copter 3.7 to merge. I am not sur the parameters are changed in 3.6
Maybe we also need a warning to explain parameters before 3.7

@Hwurzburg Hwurzburg mentioned this pull request Aug 11, 2019
Open
27 tasks
@Hwurzburg
Copy link
Contributor Author

merging

@Hwurzburg Hwurzburg merged commit 07e0a86 into ArduPilot:master Oct 23, 2019
@Hwurzburg Hwurzburg deleted the rngfndr branch October 23, 2019 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@khancyr khancyr khancyr approved these changes

@brunoolivieri brunoolivieri brunoolivieri approved these changes

@rmackay9 rmackay9 Awaiting requested review from rmackay9

Assignees
No one assigned
Labels
common-wiki DelayMerge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@Hwurzburg @khancyr @brunoolivieri