-
Notifications
You must be signed in to change notification settings - Fork 2.5k
Add substrate-interface maintenance application #2571
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
Conversation
|
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
|
I have read and hereby sign the Contributor License Agreement. |
semuelle
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @sterliakov. thanks for the application. Sounds like a good idea, and the maintainer already responded positively to your message. Let's just wait until there is a clearer agreement in place with regard to what the contributions and general maintenance will look like.
semuelle
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Happy to give it a try. Please submit your data for our KYC check while the rest of the committee reviews, @sterliakov.
|
Hi! Thank you for your approval, will submit the KYC form today. Is anything else needed from me? Any clarifications to the proposal? Aside, GitHub says "At least 5 approving reviews are required", is it a CI bug or my misunderstanding? Level 2 grants are described as requiring 3 approvals... |
This is just the maximum required for any of the levels. Unfortunately you can't set it based on the level. But we can merge it once it receives the amount necessary for each level manually. |
laboon
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. We definitely need someone to work on updating this and this seems like a very reasonable request.
|
Hi! Quite a late update, but it looks like I'm unable to convince any maintainer to actually look at my PR - I opened a few (in the main repo and in scale-codec repo) and tried to reach out by email directly, but no luck, no signs of activity anywhere. Would it make sense to migrate to a fork instead? |
|
Hey @sterliakov. That's disappointing. Feel free to submit milestone reports with links to your fork for now. If activity picks up again, we can still merge the work and delete the fork. |
Project Abstract
py-polkadot-sdk(available on PyPI assubstrate-interface) is a Python SDK for interfacing with Substrate nodes. This package powers hundreds, if not thousands, of real-world projects: 235k PyPI downloads last month, 258 GitHub stars and no signs of decreasing usage. It is the primary Python SDK for interacting with Substrate, which means it is one of the cornerstones of server-side development that needs access to Substrate information and modifications.Unfortunately, currently it is essentially unmaintained, with unit tests failing, lint violations silenced in CI, Python support lagging two versions behind, low type annotations coverage and tickets involving runtime crashes accumulating.
Grant level
Application Checklist
project_name.md).@_______:matrix.org(change the homeserver if you use a different one)