-
Notifications
You must be signed in to change notification settings - Fork 17.2k
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
Copter: merge ArduCopter.cpp and Copter.cpp #11453
Conversation
Cool, thanks |
osd.set_nav_info(nav_info); | ||
} | ||
#endif | ||
|
||
/* | ||
constructor for main Copter class | ||
*/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nitpicking: I would put the constructor as the first thing on the file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I put it down where we created the instance of it on purpose.
@rmackay9 - preferences?
* ..and many more. | ||
* | ||
* Code commit statistics can be found here: https://github.com/ArduPilot/ardupilot/graphs/contributors | ||
* Wiki: http://copter.ardupilot.org/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not holding this PR at all, but we should likely start a CONTRIBUTORS.md file, probably based on the allcontributors.org spec.
f301826
to
58ff5e5
Compare
On the dev call we decided that this was a good change for all vehicles but it's up to the vehicle maintainers to decide when they should implement this in order to reduce havoc with outstanding PRs. |
Merged, thanks! |
No description provided.