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

Install Review GitHub app and set up secrets #65

Closed
Bullrich opened this issue Jun 18, 2024 · 10 comments
Closed

Install Review GitHub app and set up secrets #65

Bullrich opened this issue Jun 18, 2024 · 10 comments

Comments

@Bullrich
Copy link
Contributor

To properly install review-bot, like requested in #62 (comment), we first need to have the GitHub app used in the runtimes repository.

As you can see in the GitHub Action Review Bot, the secrets REVIEW_APP_ID and REVIEW_APP_KEY have been installed in that repository.

We need an admin to:

  • Install the Review App with in this repository.
  • Set the REVIEW_APP_ID secret in this repository.
  • Set the REVIEW_APP_KEY secret in this repository.

This are the required steps to install Review-Bot.

@acatangiu
Copy link
Contributor

Who is admin over this org? @ggwpez @bkchr @shawntabrizi

@bkchr
Copy link
Contributor

bkchr commented Jun 27, 2024

I'm confused. This is the wrong repo? The bot should be installed for the xcm-format repo or?

@Bullrich
Copy link
Contributor Author

I'm confused. This is the wrong repo?

I understood from this comment between @acatangiu and @rzadp that xcm-format would be migrated into this repository:

they will have to migrate their RFCs from here to https://github.com/polkadot-fellows/RFCs

Was there been a different decision that is not reflected inside the issue?

@acatangiu
Copy link
Contributor

We want to run the review bot on the https://github.com/polkadot-fellows/xcm-format repo, not this repo.

(point 2 in #62 (comment))

@bkchr bkchr transferred this issue from polkadot-fellows/RFCs Jun 27, 2024
@Bullrich
Copy link
Contributor Author

Ok, I got confused by the phrasing in the next comment.

@bkchr could you migrate this issue to the xcm-format repository please?

The requirements are still the same.

Thanks!

@bkchr
Copy link
Contributor

bkchr commented Jun 27, 2024

@Bullrich can you give me the link to the docs on how to create the app ID and key? Then I get this done

@Bullrich
Copy link
Contributor Author

@pavelsupr manages the bot in Parity's org, I remember that he provided both the app id and generated a private key when we installed it in the runtimes repository.

Pavel, could you please generate a new private key and share both the key and the app id with Basti for the app Parity-Review-Bot?

Once @bkchr has these, he can set them as repository secrets and then I can port the Review-Bot configuration file.

Thanks!

@pavelsupr
Copy link

Hey @Bullrich @bkchr , in the past the ReviewBot app id and key were already issued for the Fellowship org and provided to Basti. I'll send a copy of them to Basti once again via 1Password.

@bkchr
Copy link
Contributor

bkchr commented Jun 27, 2024

Ty @pavelsupr.

@Bullrich I created the env variables as requested.

@bkchr bkchr closed this as completed Jun 27, 2024
Bullrich added a commit to Bullrich/xcm-format that referenced this issue Jun 27, 2024
Now that polkadot-fellows#65 has been resolved, review-bot can be implemented.

The following configuration has been provided in the file, but it can be changed if you disagree with the content:

### CI Files
Requires **1 dan 3** to review it and a total of **15 points** to merge.

Some possible combinations are:
- 1 dan6
- 1 dan5 & 1 dan3
- 3 dan3
- 1 dan4 & 1 dan3 & 1 dan2

### Proposals
Requires at least **2 dan 2** to review it and a total of **10 points** to merge.

Some possible combinations are:
- 1 `dan5` && any rank
- 2 `dan3`.
- 5 `dan2`.
- 1 `dan2` and 8 `dan1`.

### General files
Requires at least **2 dan 1** to review it and a total of **4 points** to merge.

Some possible combinations are:
- 1 `dan3` or above.
- 4 `dan1`.
- 2 `dan2`.
@Bullrich
Copy link
Contributor Author

Ty @pavelsupr.

@Bullrich I created the env variables as requested.

Thank you. I created #66

I tried to use more of the score system to make it more attractive for lower ranks to participate, but feel free to request any changes.

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

4 participants