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

Start on a compliance test suite for raster data providers, cleanups #60539

Merged
merged 5 commits into from
Feb 11, 2025

Conversation

nyalldawson
Copy link
Collaborator

Followup #60520. This sets in place a starting point for a raster data provider compliance suite, like our current vector data provider one.

It's neither comprehensive nor testing all providers (only the arcgismapserver and gdal ones), but it's a starting point!

@nyalldawson nyalldawson added the Cleanup Code cleanup label Feb 11, 2025
@github-actions github-actions bot added this to the 3.42.0 milestone Feb 11, 2025
Copy link

github-actions bot commented Feb 11, 2025

🪟 Windows builds

Download Windows builds of this PR for testing.
Debug symbols for this build are available here.
(Built from commit 565f1d7)

🪟 Windows Qt6 builds

Download Windows Qt6 builds of this PR for testing.
(Built from commit 565f1d7)

Copy link
Contributor

@rouault rouault left a comment

Choose a reason for hiding this comment

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

Were the equality operators changes intended to go through that PR ?

@nyalldawson
Copy link
Collaborator Author

@rouault

Were the equality operators changes intended to go through that PR ?

They were needed so that the test of provider Uri equality after cloning works

@nyalldawson nyalldawson merged commit 38be0a3 into qgis:master Feb 11, 2025
39 checks passed
@nyalldawson nyalldawson deleted the raster_dp branch February 11, 2025 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Cleanup Code cleanup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants