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

add driver for Fenix device #1405

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Conversation

markus2marks
Copy link

Your PR should:

  • all description should be in the commit messages (no text in the pr)
  • your pr should be rebased on the current HEAD
  • one commit per atomic feature (every commit should build)
  • no fixup commits
  • pass all the compile and CI targets

For coding, style guide, architecture information please see our development guide:
https://xcsoar.readthedocs.io/en/latest/index.html

@markus2marks markus2marks requested a review from a team as a code owner April 17, 2024 19:49
@markus2marks
Copy link
Author

it's the first time for me that i add something in XCSoar, how can i start the CI tests?

@lordfolken
Copy link
Contributor

lordfolken commented Apr 17, 2024

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.

@markus2marks
Copy link
Author

can you start the CI again please?

@lordfolken
Copy link
Contributor

lordfolken commented Apr 19, 2024

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.
Please respect code and comment style.
Please dont duplicate code. Import stuff from the class of lxnav or better yet structure those commits centrally.
Things like cout are not acceptable.

In general your git commits should be like a a Lego kit. Corrections etc I don't want in the git history.

@markus2marks
Copy link
Author

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?
I would only use the LXWP1 function from the LX driver

@lordfolken
Copy link
Contributor

First get the PR in order. Dont merge rebase your commits.

@MaxKellermann
Copy link
Contributor

strncpy, wtf!

@markus2marks
Copy link
Author

quick and dirty ;) whats your suggestion for it?

@lordfolken lordfolken marked this pull request as draft April 28, 2024 19:18
@lordfolken
Copy link
Contributor

Besides the code issue, we need a clean git history.
This article explains how to rebase and push code to a branch
https://docs.gitlab.com/ee/topics/git/git_rebase.html
Maybe this helps you?

@markus2marks
Copy link
Author

Yes, I will clean it up when I have time, at the moment I'm busy with my work ;)

@markus2marks markus2marks reopened this May 5, 2024
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.

None yet

3 participants