Skip to content

🚨 Check building for docs.rs#75

Merged
weiji14 merged 2 commits intomainfrom
ci/docs_rs
Mar 4, 2026
Merged

🚨 Check building for docs.rs#75
weiji14 merged 2 commits intomainfrom
ci/docs_rs

Conversation

@weiji14
Copy link
Owner

@weiji14 weiji14 commented Mar 3, 2026

Ensure that documentation build will work for docs.rs at release time. Uses cargo-docs-rs running on Rust nightly.

Alternatively, run the following command locally:

cargo +nightly doc --all-features --config 'build.rustdocflags=["--cfg", "docsrs"]'
# or
cargo +nightly doc --all-features --config 'build.rustdocflags=["--cfg", "docsrs"]' --no-deps -Zrustdoc-map

Also updating a bunch of dependencies to fix the fallout from rust-lang/rust#138907, though a couple still remain.

Ensure that documentation build will work for docs.rs at release time. Uses cargo-docs-rs.
@weiji14 weiji14 added this to the 0.1.0 milestone Mar 3, 2026
@weiji14 weiji14 self-assigned this Mar 3, 2026
@codspeed-hq
Copy link

codspeed-hq bot commented Mar 3, 2026

Merging this PR will degrade performance by 29.64%

⚡ 1 improved benchmark
❌ 4 (👁 4) regressed benchmarks
✅ 2 untouched benchmarks

Performance Changes

Benchmark BASE HEAD Efficiency
2_image-tiff_CPU[Sentinel-2 TCI] 5.2 s 5 s +5.24%
👁 test_read_geotiff_multi_band 26.3 ms 31.9 ms -17.58%
👁 test_xarray_backend_open_dataarray[rasterio] 28.9 ms 34.5 ms -16.38%
👁 test_xarray_backend_open_dataarray[cog3pio] 50.5 ms 56.2 ms -10.04%
👁 test_read_geotiff_remote 13.4 ms 19.1 ms -29.64%

Comparing ci/docs_rs (6a7a94d) with main (59d7672)

Open in CodSpeed

Update a whole bunch of transitive dependencies to ensure they can build on rust nightly. Still a couple remaining though.
@weiji14 weiji14 marked this pull request as ready for review March 4, 2026 04:29
@weiji14 weiji14 merged commit 04fb589 into main Mar 4, 2026
26 checks passed
@weiji14 weiji14 deleted the ci/docs_rs branch March 4, 2026 04:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant