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

Release into PyPI #22

Closed
esteve opened this issue Apr 19, 2022 · 4 comments · Fixed by #41
Closed

Release into PyPI #22

esteve opened this issue Apr 19, 2022 · 4 comments · Fixed by #41

Comments

@esteve
Copy link
Contributor

esteve commented Apr 19, 2022

Once we're happy with the feature set, we should release colcon-cargo into PyPI

Particularly, when colcon-cargo is fully integrated into colcon test, it would be a good time for a release

@stelzo
Copy link

stelzo commented Jul 9, 2024

It would be great to publish it using SemVer to indicate feature parity.

A published package would make it possible to integrate Cargo dependencies into rosdep (ros-infrastructure/rosdep#923), which could greatly reduce Rust related overhead for non-Rust-speaking users that just want to build a binary package.

@luca-della-vedova
Copy link
Member

Hi @stelzo!

I had a chat with the maintainers and they mentioned that they would like to see at least better coverage and testing in the library before publishing it, to show that the library is working as intended (which is why I've been taking that direction with #35 and #39

@luca-della-vedova
Copy link
Member

Everything is unblocked now, at least on the colcon-cargo side, I'm happy to start making the first tagged release (0.1.0?) unless anyone wants to take ownership of that @esteve, @jhdcs, @mxgrey.

colcon-ros-cargo is still pending a release on crates.io of cargo-ament-build and colcon/colcon-ros-cargo#22

@esteve
Copy link
Contributor Author

esteve commented Aug 26, 2024

@luca-della-vedova I've just published a new release of cargo-ament-build

I can do releases for colcon-cargo and colcon-ros-cargo once you think they are ready. Thanks for all the work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

3 participants