Skip to content

Comments

rasterio.merge.merge: fix formatting of pre-defined method docs#3340

Merged
snowman2 merged 1 commit intorasterio:mainfrom
adamjstewart:docs/merge
May 20, 2025
Merged

rasterio.merge.merge: fix formatting of pre-defined method docs#3340
snowman2 merged 1 commit intorasterio:mainfrom
adamjstewart:docs/merge

Conversation

@adamjstewart
Copy link
Contributor

@adamjstewart adamjstewart commented May 19, 2025

Fixes the formatting of the "pre-defined method" section of the merge docs.

Before

Screenshot 2025-05-19 at 8 18 19 PM

After

Screenshot 2025-05-20 at 9 54 46 AM

Copy link
Member

@snowman2 snowman2 left a comment

Choose a reason for hiding this comment

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

Thanks @adamjstewart 👍

@adamjstewart
Copy link
Contributor Author

@adamjstewart
Copy link
Contributor Author

@sgillies
Copy link
Member

@adamjstewart we (mostly me) haven't been able to keep up with recent GDAL releases, which change warp output and also trigger a warning that we don't handle (it's turned into an error). If this PR doesn't change code execution, I might merge it regardless.

@adamjstewart
Copy link
Contributor Author

I still want to double check if the docs are now formatting correctly, let me at least fix the docs before merging.

I probably can't help too much with GDAL churn, but I can help keep the docs building cleanly. I'll see if I can fix enough warnings/errors that we can enable "fail-on-warning" for the docs build.

@adamjstewart adamjstewart marked this pull request as ready for review May 20, 2025 07:55
@adamjstewart
Copy link
Contributor Author

Okay, can confirm that the docs look good now, added a screenshot above. Should be ready to merge.

@snowman2 snowman2 merged commit 03a7097 into rasterio:main May 20, 2025
7 of 22 checks passed
@adamjstewart adamjstewart deleted the docs/merge branch May 20, 2025 12:12
@snowman2 snowman2 added the backport-1.4 To be backported to maint-1.4 label Nov 6, 2025
@snowman2 snowman2 added this to the 1.4.4 milestone Nov 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-1.4 To be backported to maint-1.4 bug documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants