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

fix some coverity tests #11291

Merged
merged 1 commit into from
Nov 17, 2024
Merged

Conversation

jjimenezshaw
Copy link
Contributor

What does this PR do?

Fix some coverity scan tests that were failing

  • Make sure code is correctly formatted (cf pre-commit configuration)
  • Add test case(s)
  • Review
  • Adjust for comments
  • All CI builds and checks have passed

@coveralls
Copy link
Collaborator

Coverage Status

coverage: 73.701%. remained the same
when pulling 6c4153f on jjimenezshaw:fix-coverity-tests
into 9b5cc40 on OSGeo:master.

@rouault rouault merged commit d9614a0 into OSGeo:master Nov 17, 2024
35 of 36 checks passed
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.

3 participants