-
Notifications
You must be signed in to change notification settings - Fork 273
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
add driver for Fenix device #1405
base: master
Are you sure you want to change the base?
Conversation
it's the first time for me that i add something in XCSoar, how can i start the CI tests? |
As its your first time a project member has to approve them. |
can you start the CI again please? |
Just for the record the macosx failure is in master And not your PR. Please cleanup your git history with proper formatted commits and commit messages. In general your git commits should be like a a Lego kit. Corrections etc I don't want in the git history. |
Most of the functions of the LX driver are static, I cannot import them. Should the Fenix driver inherit the functions of the LX driver? or what are your suggestions to solve this problem? |
First get the PR in order. Dont merge rebase your commits. |
strncpy, wtf! |
quick and dirty ;) whats your suggestion for it? |
Besides the code issue, we need a clean git history. |
Yes, I will clean it up when I have time, at the moment I'm busy with my work ;) |
f0b6b82
to
af44095
Compare
Your PR should:
For coding, style guide, architecture information please see our development guide:
https://xcsoar.readthedocs.io/en/latest/index.html