Skip to content

ci: release-plz#26

Open
flying-sheep wants to merge 8 commits into
mainfrom
pa/release-plz
Open

ci: release-plz#26
flying-sheep wants to merge 8 commits into
mainfrom
pa/release-plz

Conversation

@flying-sheep
Copy link
Copy Markdown
Member

@flying-sheep flying-sheep commented May 20, 2026

https://release-plz.dev/ is the most convenient way to release Rust crates:

  1. it maintains a perma-PR that maintains a changelog from conventional commits.
  2. when you merge it, it makes releases for all changed crates.
  3. the first commit to main after merging it opens a new one.

@flying-sheep flying-sheep requested a review from ilan-gold May 20, 2026 08:39
Copy link
Copy Markdown
Collaborator

@ilan-gold ilan-gold left a comment

Choose a reason for hiding this comment

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

Image We should put this info in the `README.md`, prehaps. Also see above photo - seems like a nice win for green checkmarks

Comment thread .github/workflows/release.yml Outdated
@flying-sheep flying-sheep requested a review from ilan-gold May 20, 2026 09:06
Comment thread pyanndata/README.md
Comment on lines +1 to +7
pyanndata
=========

[PyO3] powered Python for the [anndata] crate.

[PyO3]: https://pyo3.rs/
[anndata]: https://crates.io/crates/anndata
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think that this one should be left out as Kai has the separate snapatac2 branch for it

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 think that this one should be left out as Kai has the separate snapatac2 branch for it

Yes, unless you think it makes more sense to add it to the ecosystem.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Yes, unless you think it makes more sense to add it to the ecosystem.

I suppose it can't hurt! https://github.com/scverse/SnapATAC2/blob/main/Cargo.toml#L22-L27 It looks like everything is pinned anyway so I guess it's safe

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Awesome, let’s do it!

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