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

Apoint more maintainers #111

Open
teskje opened this issue Jul 13, 2020 · 5 comments
Open

Apoint more maintainers #111

teskje opened this issue Jul 13, 2020 · 5 comments
Labels
help wanted Extra attention is needed

Comments

@teskje
Copy link
Collaborator

teskje commented Jul 13, 2020

Lately this project seems to be quite undermaintained. There are a couple of PRs open currently that would be useful to have merged, but they don't get reviewed.

I think that is a problem, since the stm32f3xx-hal is usually how beginners in embedded Rust (like me) have their first contact with the ecosystem (due to the great Discovery book). Bugs and missing features of this crate can lead to needlessly steep learning curves, so it would be good if we could fix those in a timely manner.

I am sure the current maintainers have a busy life and I certainly don't want to blame anyone! But I think we need to find a solution to this problem. The most obvious one would be giving maintainer rights to more people, to share the load of getting PRs reviewed and merged.

@dfrankland
Copy link
Member

dfrankland commented Jul 13, 2020

I agree! Folks who'd like to help maintain, please feel free to reach out to me and I can get you setup 👍

@Sh3Rm4n
Copy link
Member

Sh3Rm4n commented Jul 14, 2020

Thanks for taking action!

I'm really sorry for being silent in the last few months without notice. I was busy with my internship and quite overwhelmed by the PR's and took a step back.

I'm a rust beginner myself and I lack the experience too confidently guide the current PR's to the right design decision. In the end I always looked at [stm32f1xx-hal] or [stm32f4xx-hal] how they've done it, because there is where the innovation takes place. I haven't had the time to give constructive feedback besides nitpicking on some points.
It ended up that if I had the spare-time and motivation to do some embedded rust, I felt the pressure to take care of this repository (as many are still waiting on my feedback). But I really lack the experience to help productively, as I'm missing the time to play around with this stuff myself.

If more people would work on the repository, it would help enormously

I think that is a problem, since the stm32f3xx-hal is usually how beginners in embedded Rust (like me) have their first contact with the ecosystem (due to the great Discovery book). Bugs and missing features of this crate can lead to needlessly steep learning curves, so it would be good if we could fix those in a timely manner.

Strongly agree on that point! There are some points I would like to improve to smoothen the experience for beginners. I'd wish for better alignments between the stm32-rs hal's on implementation design for some modules for more consistent interfaces overall,
(E.g this issue comes to my mind). Some "does" and "don'ts" or rather best practices would help. I think focusing on the API design is the most important. Implementation details can always be improved with time.
But this could be discussed in the new stm32-rs/meta repo to open this discussion up to everyone.

I also think that II'll be able to get some work done on some smaller PRs and issues in the next few days again. :)

@Sh3Rm4n Sh3Rm4n pinned this issue Jul 14, 2020
@Sh3Rm4n Sh3Rm4n added the help wanted Extra attention is needed label Jul 14, 2020
@teskje
Copy link
Collaborator Author

teskje commented Jul 14, 2020

@Sh3Rm4n No worries! And thanks for the effort you have put in so far!

@dfrankland has given me contributor rights to this repo now too, so I will be able to help out in the future. Having more people would be good of course. I can't (or rather: shouldn't) review my own PRs ;)

@Sh3Rm4n Sh3Rm4n unpinned this issue Dec 9, 2020
@Sh3Rm4n Sh3Rm4n pinned this issue Mar 28, 2021
@seanybaggins
Copy link
Contributor

seanybaggins commented Sep 26, 2021

Hi @Sh3Rm4n, @dfrankland, and @ra-kete . I am currently in the job market. Hoping to get a job that is remote, works with embedded, and utilizes Rust. I would love to help out with maintenance while I am looking for a job.

@Sh3Rm4n
Copy link
Member

Sh3Rm4n commented Oct 4, 2021

Hey @seanybaggins, I'm happy you are interested to help and wanting to contribute to stm32f3xx-hal! :)

I sadly had not much time in the last weeks to make progress on some PRs I'm working on, so many of these are still WIP. And there are also different issues, which needs to be tackled.

But I suggest you just go through these, find things you are interested in solving or even find a peripheral, which is currently not yet supported and draft possible implementations.

You can always open PRs, and I'll try my best to review them. And you can ask me anything, if you have any questions :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants