diff --git a/.github/review-bot.yml b/.github/review-bot.yml index 230f2d0ca5..7f81f6a3c4 100644 --- a/.github/review-bot.yml +++ b/.github/review-bot.yml @@ -9,12 +9,9 @@ rules: - name: Relay and system files condition: include: - - ^relay\/kusama\/.* - - ^relay\/polkadot\/.* + - ^relay\/.* - ^system-parachains\/.* - - ^CHANGELOG$ - exclude: - - ^relay\/.+\.adoc$ + - ^CHANGELOG.md$ type: fellows minRank: 3 minApprovals: 4 @@ -22,11 +19,10 @@ rules: condition: include: - '.*' - exclude: - - ^relay\/kusama\/.* - - ^relay\/polkadot\/.* - - ^\.github/.* + exclude: + - ^relay\/.* - ^system-parachains\/.* - - ^target\/.* + - ^\.github/.* + - ^CHANGELOG.md$ type: fellows minRank: 2 diff --git a/README.md b/README.md index 8b01afe128..e24a7e40db 100644 --- a/README.md +++ b/README.md @@ -25,6 +25,16 @@ Each leaf folder contains one runtime crate: └── glutton-kusama ``` +## Approval rights + +The approval rights are configured in [`review-bot.yml`](.github/review-bot.yml). The rights are configured as: + +- All files in `.github` require two approvals from Fellowship members of rank 4 or higher. +- `CHANGELOG.md`, `relay/*` or `system-parachains/*` require four approvals from Fellowship members of rank 3 or higher. +- All other files require the approval from one Fellowship member of rank 2 or higher. + +The review-bot uses the on-chain identity to map from a GitHub account to a Fellowship member. This requires that each Fellowship member add their GitHub handle to their on-chain identity. Check [here](docs/on-chain-identity.md) for instructions. + # Working on Pull Requests To merge a pull request, we use [Auto Merge Bot](https://github.com/paritytech/auto-merge-bot). diff --git a/docs/on-chain-identity-process.png b/docs/on-chain-identity-process.png new file mode 100644 index 0000000000..fb08c7e6c1 Binary files /dev/null and b/docs/on-chain-identity-process.png differ diff --git a/docs/on-chain-identity.md b/docs/on-chain-identity.md new file mode 100644 index 0000000000..014a2bb444 --- /dev/null +++ b/docs/on-chain-identity.md @@ -0,0 +1,15 @@ +# Setting up on-chain identity + +As a member of the Polkadot Technical Fellowship you should set an on-chain identity for your account on the Polkadot relay chain. Your identity should also include a field `github` pointing to your github username. Unfortunately this is a custom field that isn't supported by the polkadot.js UI, therefore in order to set it you'll need to create the identity extrinsic yourself. + +In the polkadot.js UI go to `Developer > Extrinsics`. Select your fellowship member account and pick the extrinsic `identity.setIdentity`. Below is an example screenshot of what it should look like including the `github` field. + +![Example extrinsic to set on-chain identity](on-chain-identity-process.png) + +Keep in mind that when filling in the data with `Raw` type only ASCII is accepted, therefore if you need to use UTF-8 characters (e.g. diacritics or emojis), you should convert the data to hex beforehand. For example, my legal name is "André Silva", which I need to submit as `0x416e6472c3a92053696c7661`. You can use [this](https://onlinehextools.com/convert-utf8-to-hex) tool to do the conversion, just disable the options `Add Hex Base` and `Space Between Hex Values`, after the conversion you should prepend the result with a single `0x` and submit as `Raw`. + +After submitting the extrinsic you won't be able to see the `github` field in the polkadot.js UI (custom fields aren't shown). You can confirm that everything was set correctly by checking the state (`Developer > Chain State > identity.identityOf`), or by using a block explorer like [Statescan](https://polkadot.statescan.io) which shows custom fields. + +## Getting a judgment on your identity + +After your identity is properly set you can get a judgment on it from one of the existing registrars. For instructions on how to get a judgment from the W3F registrar check [this](https://registrar.web3.foundation/) page which contains links to instructions. Due to the custom field in your identity the automated verification process won't work and you'll need to join the support channel on matrix to complete the verification. diff --git a/relay/polkadot/README.adoc b/relay/polkadot/README.adoc deleted file mode 100644 index 3337331081..0000000000 --- a/relay/polkadot/README.adoc +++ /dev/null @@ -1,5 +0,0 @@ - -= Polkadot Runtime - -placeholder -//TODO Write content :) (https://github.com/paritytech/polkadot/issues/159)