Skip to content
This repository was archived by the owner on Jul 5, 2024. It is now read-only.

chore(github CI): use pull_request_target event for external reveiwers#479

Closed
0xmountaintop wants to merge 1 commit into
privacy-ethereum:mainfrom
scroll-tech:github
Closed

chore(github CI): use pull_request_target event for external reveiwers#479
0xmountaintop wants to merge 1 commit into
privacy-ethereum:mainfrom
scroll-tech:github

Conversation

@0xmountaintop
Copy link
Copy Markdown
Collaborator

No description provided.

@0xmountaintop 0xmountaintop requested review from a team and ChihChengLiang and removed request for a team April 28, 2022 03:05
@github-actions github-actions Bot added the CI Issues related to the Continuous Integration mechanisms of the repository. label Apr 28, 2022
@0xmountaintop
Copy link
Copy Markdown
Collaborator Author

0xmountaintop commented Apr 28, 2022

Hmmm not sure how to trigger "Reviewer lottery" action (as in #467) to verify this change works.

@DreamWuGit
Copy link
Copy Markdown
Collaborator

@HAOYUatHZ I got it in #481 again, perhaps I can help you to verify this. :)

@CPerezz
Copy link
Copy Markdown
Contributor

CPerezz commented May 3, 2022

It was decided to remove this workflow for now and simply ping Scroll members for PR reviews. As there's no way to overpass the permission system for PR setup..

I suggest we close this for now and we don't loose more time :)

@0xmountaintop 0xmountaintop deleted the github branch May 3, 2022 10:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CI Issues related to the Continuous Integration mechanisms of the repository.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants