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

implemented fcu::flightcontroller::isConnected member. #37

Merged
merged 1 commit into from
Jun 2, 2019
Merged

implemented fcu::flightcontroller::isConnected member. #37

merged 1 commit into from
Jun 2, 2019

Conversation

Moldisocks
Copy link
Contributor

Super small addition, just added the implementation for the already defined member fcu::flightcontroller::isConnected. Hopefully i will be able to contribute more the the project in the future.

@christianrauch
Copy link
Owner

Thanks.
Can you make the method const?

@christianrauch
Copy link
Owner

Sorry, there need to be more methods declared as const before this one can be const too.
Just undo the last commit and I will merge this.
I will later deal with declaring methods as const.

@Moldisocks
Copy link
Contributor Author

Hey mate, i've undone the last commit. Apologies for the wait, it's my first time with PRs and i'm a git newbie in general, was just trying to figure it out. Thanks for the patience.

@christianrauch christianrauch merged commit ca9f3f0 into christianrauch:master Jun 2, 2019
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