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

upgraded review bot to v2.1.0 #64

Merged
merged 1 commit into from
Oct 18, 2023

Conversation

Bullrich
Copy link
Contributor

Upgraded to version 2.1.0 which has paritytech/review-bot#94, a change in the logic of the action to overcome some problems with permissions coming from PRs from forks

For this, we needed to divide the actions into two files:

  • A first action that triggers on PRs and reviews and uploads the PR number.
  • A second action which is triggered under the completion of the first one and runs as the action normally runs (but won't have any problems regarding permissions because it is triggered from the master branch)

Upgraded to version 2.1.0 which has paritytech/review-bot#94, a change in the logic of the action to overcome some problems with permissions coming from PRs from forks

For this, we needed to divide the actions into two files:
- A first action that triggers on PRs and reviews and uploads the PR number.
- A second action which is triggered under the completion of the first one and runs as the action normally runs (but won't have any problems regarding permissions because it is triggered from the master branch)
@Bullrich
Copy link
Contributor Author

/merge

@github-actions
Copy link

There was a problem running the action.

❌😵❌

Please find more information in the logs.

@Bullrich
Copy link
Contributor Author

There was a problem running the action.

❌😵❌

Please find more information in the logs.

This is unexpected, but welcome.

It seems that a GitHub action that would modify a workflow file can not do it by default (which, for this particular case, makes sense to keep it).

For normal code it should work without any issues.

@bkchr bkchr merged commit b641df5 into polkadot-fellows:main Oct 18, 2023
5 of 6 checks passed
@Bullrich Bullrich deleted the bullrich/review-bot/upgrade branch October 18, 2023 10:28
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

Successfully merging this pull request may close these issues.

2 participants