Skip to content

dev/release: Simplify cargo publish#1025

Open
ajaypadwal73 wants to merge 4 commits into
apache:mainfrom
ajaypadwal73:issue-702-simplify-cargo-publish
Open

dev/release: Simplify cargo publish#1025
ajaypadwal73 wants to merge 4 commits into
apache:mainfrom
ajaypadwal73:issue-702-simplify-cargo-publish

Conversation

@ajaypadwal73

@ajaypadwal73 ajaypadwal73 commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a cargo dependency graph checker and derive publish order from cargo metadata
  • split random data generation, base test helpers, and raster testing helpers out of sedona-testing
  • move raster sd_format handling into sedona-raster-functions and register its kernel from sedona
  • relocate pointcloud integration tests to the top-level sedona crate

Fixes #702

Verification

  • cargo fmt
  • python3 dev/release/check-cargo-dependencies.py --print-publish-order
  • cargo check/test sweeps run locally during review

@ajaypadwal73 ajaypadwal73 force-pushed the issue-702-simplify-cargo-publish branch from 8a44c2e to 446d1da Compare July 4, 2026 19:18

@paleolimbot paleolimbot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I will need a bit of time to look this over carefully, but in the meantime can you add a run of check-cargo-dependencies.py to one of the CI runs? Possibly the packaging run, or rust.yml with a new matrix name ("chekc-publish"?). Then when I review I'll know it works!

@ajaypadwal73

Copy link
Copy Markdown
Contributor Author

@paleolimbot Done, added a check-publish matrix job in rust.yml that runs dev/release/check-cargo-dependencies.py --print-publish-order. The new check is passing on this PR.

@ajaypadwal73 ajaypadwal73 force-pushed the issue-702-simplify-cargo-publish branch from d812d91 to cab248d Compare July 15, 2026 00:14
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.

dev/release: Simplify cargo publish

2 participants