Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

FRAME: add a basic readme #13520

Merged
merged 1 commit into from Mar 4, 2023
Merged

FRAME: add a basic readme #13520

merged 1 commit into from Mar 4, 2023

Conversation

ghost
Copy link

@ghost ghost commented Mar 3, 2023

Streamlining dev processes starts with trivial things - like a readme.

Context

https://forum.polkadot.network/t/ecosystem-wide-frame-project-manager/1856

calling out for the ecosystem to put forth and/or agree upon a candidate to do this task, and have this person also be funded directly by the treasury, not Parity. With this person being outside of Parity, I hope that we can achieve a higher degree of uniformity across pallet repositories such as Substrate and ORML etc.

No need for inaction during the wait.

I would like to do some basic work around FRAME, thus the future project-manager can be more efficient.

My work would involve mostly the technical stuff (repository, issue-trackers, automations etc.). At some point I would "dive deep", e.g. providing abstractions to reduce duplications / dev-effort etc.

But it all starts with the basics - like a tiny subsystem readme.

To speed things up (and avoid "time evaporated" on both sides) I'm asking here for a budget (via bounty/tip) and I would start (or better: continue) right away.

Polkadot address: 13mDFYievQov7ie1257v9bXHDyLvpXWPrgYopnUjYxgD9DnZ

@cla-bot-2021
Copy link

cla-bot-2021 bot commented Mar 3, 2023

User @abebeos, please sign the CLA here.

@KiChjang KiChjang added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit labels Mar 3, 2023
@Polkadot-Forum
Copy link

This pull request has been mentioned on Polkadot Forum. There might be relevant details there:

https://forum.polkadot.network/t/ecosystem-wide-frame-project-manager/1856/9

@xlc
Copy link
Contributor

xlc commented Mar 3, 2023

Happy to see you are willing to help. However add a simple readme file isn't justifiable for tips, which are usually done for existing work.

If you do believe your prior contribution is significant enough, you should apply it properly to the Polkadot treasury.

If you want upfront payment before you start contribution, you would need to make a proper treasury proposal and convince the community first.

(I don't have permission to trigger the tip bot and I am also just an external contributor)

@ghost
Copy link
Author

ghost commented Mar 3, 2023

However add a simple readme file isn't justifiable for tips,

It should be, given the context.

If you do believe your prior contribution is significant enough, you should apply it properly to the Polkadot treasury.

I think i'd better not apply...

make a proper treasury proposal and convince the community first.

Me already in streamlining-mode. (redundant )Bureaucracy would kill my flow...

(All good, ty for the info, will for now just go on with the tiny steps)

@bkchr
Copy link
Member

bkchr commented Mar 3, 2023

Yeah @xlc is right. We don't use tips for regular payments, they are just tips as the name implies. We also don't pay upfront any tips, because we don't want to attract people that are only making noise and costing time. If you want to have proper payments, it would need to be done through treasury as @xlc said.

@ghost
Copy link
Author

ghost commented Mar 4, 2023

Two things:

  • someone please review/merge
  • someone please trigger a minitip, so i can quickly verify the process (without the treasury step).

Except you think that i "only making noise and costing time".

@ghost
Copy link
Author

ghost commented Mar 4, 2023

Alternatively use this issue #13522 (for a mini tip)

@bkchr
Copy link
Member

bkchr commented Mar 4, 2023

  • someone please trigger a minitip, so i can quickly verify the process (without the treasury step).

I don't get why you want to verify the process? Verify for what? The payment will not be instant as the council still needs to accept it and decide on the value.

@ghost
Copy link
Author

ghost commented Mar 4, 2023

I don't get why you want to verify the process? Verify for what?

I don't get it myself (analysis runs non-consciously)

@ggwpez
Copy link
Member

ggwpez commented Mar 4, 2023

bot merge

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot paritytech-processbot bot merged commit f57ff37 into paritytech:master Mar 4, 2023
ukint-vs pushed a commit to gear-tech/substrate that referenced this pull request Apr 10, 2023
nathanwhit pushed a commit to nathanwhit/substrate that referenced this pull request Jul 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants