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

Terrain: Cleanup Terrain Queries #11797

Merged
merged 1 commit into from
Sep 6, 2024
Merged

Conversation

HTRamsey
Copy link
Collaborator

@HTRamsey HTRamsey commented Aug 21, 2024

Clean up and better structure Terrain files in prep for #11795
Fixes some QObject parenting
Finishes setting up unit tests

@HTRamsey HTRamsey force-pushed the dev-terrainquery branch 2 times, most recently from a36cfaa to 1faadb6 Compare August 22, 2024 06:55
@HTRamsey HTRamsey marked this pull request as ready for review August 22, 2024 06:56
@DonLakeFlyer
Copy link
Contributor

Are you running some sort of lint thing on stuff before you checkin? Removing tabs and moving around the foo* bar to foo *bar generates all sorts of unnecessary diffs in the commit.

@HTRamsey
Copy link
Collaborator Author

I don't but it would be really nice to get everything standardized because with so many different contributors things have gotten to be a mess so I wanted to do it right and clean when I work on things. But yeah I get the git diffs part which is why I try to add unit tests to everything I do to help verify changes. I don't think a formatter should be run on everything at once, but rather only do it on one thing at a time that's been verified through testing.

However with these files it's a bit different because they have to change drastically anyways to cleanly implement different providers.

@HTRamsey HTRamsey force-pushed the dev-terrainquery branch 2 times, most recently from aa20c15 to 63cc141 Compare September 6, 2024 15:42
@DonLakeFlyer
Copy link
Contributor

I don't but it would be really nice to get everything standardized because with so many different contributors things have gotten to be a mess so I wanted to do it right and clean when I work on things.

Ok, all good then. I'll change to be closer to what most people like.

Copy link
Contributor

@DonLakeFlyer DonLakeFlyer left a comment

Choose a reason for hiding this comment

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

Are you going to pull out the old AirMap stuff in separate pull?

@HTRamsey
Copy link
Collaborator Author

HTRamsey commented Sep 6, 2024

Are you going to pull out the old AirMap stuff in separate pull?

That or convert it to Copernicus if I can. Not sure why some stuff was converted to Copernicus but not everything.

@HTRamsey HTRamsey merged commit 39fab5b into mavlink:master Sep 6, 2024
8 checks passed
@HTRamsey HTRamsey deleted the dev-terrainquery branch September 6, 2024 21:04
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