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

Please allow others to manage this repo if you do not have time to properly review PRs and Issues! #168

Open
seenickcode opened this issue Mar 21, 2021 · 9 comments

Comments

@seenickcode
Copy link

@wyyerd it may be a good idea to give certain contributors write access to this repo so we can ensure it's maintained promptly. CC @seanpianka

@seanpianka
Copy link
Collaborator

cc: @kestred @stearnsc

@arlyon
Copy link
Collaborator

arlyon commented Mar 28, 2021

I have been maintaining a fork at https://github.com/arlyon/async-stripe for a while now, it's diverged a little as I've been adding other things but I have the time to stay on top of issues (and very open to PRs!)

The most notable feature is pluggable runtimes so you can use it on tokio and async-std, here is a more comprehensive list: arlyon/async-stripe#42

@stearnsc
Copy link
Member

Hi! I'm sorry, this repo had flown somewhat under my radar (I inherited it via reorganization at work). I'm absolutely in favor of adding more maintainers. I believe @kestred is still the only one who can publish versions, but that can probably change as well. @seanpianka are you interested in this? @arlyon is upstreaming your fork something you're interested in doing?

I don't have a lot of context into whether there are particular concerns I should be considering in adding maintainers re planned direction of the repository, but my instinct is to just add one or both of you as maintainers, if you're interested. @kestred do you have any thoughts or concerns?

@arlyon
Copy link
Collaborator

arlyon commented Mar 29, 2021

Happy to upstream any changes. There are about 70 commits at this point, so there's a lot that I have changed but the link above has a short summary which I believe is for the better so it's really down to how much you are willing to take on. It's foss for a reason! :)

Some of the more opinionated changes are moving to github actions, reworking the feature flags, changing the tests (mainly to utilise github actions better), changing the error code, using cargo make, and adding sorting and grouping to rustfmt. Obviously if any one of those is a dealbreaker it makes the code harder to upstream but as time went on keeping compatibility with the original fork was less and less of a priority. Let me know what you think.

@kestred
Copy link
Collaborator

kestred commented Mar 31, 2021

@stearnsc, can you choose a committer you trust to maintain this and I can transfer the Cargo authorization to you to assign as needed?

@seanpianka
Copy link
Collaborator

Perhaps both of us, I'm up for helping get important fixes merged and start to upstream @arlyon's changes.

@arlyon
Copy link
Collaborator

arlyon commented Apr 1, 2021

Happy to help with the upstreaming on this side as well. It would reduce the bottleneck significantly having multiple people but not critical.

@stearnsc
Copy link
Member

stearnsc commented Apr 6, 2021

Alright, @seanpianka and @arlyon have both been added as maintainers. @kestred can you add me as an owner via cargo?

@mikeumus
Copy link

@kestred @stearnsc @arlyon @seanpianka may we consider turning on GitHub Discussions to get a little more organized and GitHub Sponsors to help water the repo a little? water 💦 🌱 🌞

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

No branches or pull requests

6 participants