rust-embedded / wg

Coordination repository of the embedded devices Working Group
1.91k stars 98 forks source link

Transfer heapless and volatile-register to libs team #710

Closed adamgreig closed 11 months ago

adamgreig commented 11 months ago

I propose (with @japaric's blessing!) transferring heapless and volatile-register repos to the WG under the libs team. Heapless is widely used across the embedded ecosystem but the current maintainers don't have much time for updates, while volatile-register is used by cortex-m and we'd like to at least fix a soundness issue for us and other users (https://github.com/rust-embedded/cortex-m/issues/485). The newly formed libs team is a perfect fit, not coincidentally.

I hope we'll have a few more members soon, now that the team exists. Still, the current libs team should vote:

Dirbaio commented 11 months ago

I vote yes :laughing:

newAM commented 11 months ago

I vote yes! ♥️ heapless

japaric commented 11 months ago

I propose (with @japaric's blessing!) transferring heapless and volatile-register repos to the WG under the libs team

I'm on board :+1: (let me know when / where to transfer the repos)

Dirbaio commented 11 months ago

I don't think there's any blocker left? you can transfer to https://github.com/rust-embedded, @adamgreig will have to configure the permissions I guess.

adamgreig commented 11 months ago

I'm on board 👍 (let me know when / where to transfer the repos)

Thanks! Just transfer them to rust-embedded any time, I'll then assign them to the libs team.

japaric commented 11 months ago

done!

adamgreig commented 11 months ago

Thank you! I've put them under the libs team now. Could you add the team on crates.io as well, please?

cargo owner --add github:rust-embedded:libs
adamgreig commented 11 months ago

@rust-embedded/libs, there's some notes in https://github.com/rust-embedded/wg/blob/master/ops/post-transfer.md about the changes to make, which for this I think boil down to:

japaric commented 11 months ago

@adamgreig it seems like I need to be temporarily added to the libs team to do that:

$ cargo owner --add github:rust-embedded:libs
error: failed to invite owners to crate `heapless` on registry at https://crates.io

Caused by:
  the remote server responded with an error: only members of a team or organization owners can add it as an owner
adamgreig commented 11 months ago

Huh, I didn't know that! I wonder if it's a recent change. Could you add me as an owner directly instead, then? At least that also avoids not being able to update owners in future, and I should be able to add the libs team.

japaric commented 11 months ago

@adamgreig crates.io invites sent!

adamgreig commented 11 months ago

Thanks, all sorted!

Dirbaio commented 11 months ago

all done! thanks guys :partying_face: