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

[infra] Create first reusable workflow #159

Merged

Conversation

michelengelen
Copy link
Member

Adds the first reusable workflow: add PR reviewers based on team

@michelengelen michelengelen added the scope: infra Org infrastructure work going on behind the scenes label May 6, 2024
@michelengelen michelengelen self-assigned this May 6, 2024
Co-authored-by: Jose C Quintas Jr <[email protected]>
Signed-off-by: Michel Engelen <[email protected]>
@michelengelen
Copy link
Member Author

FYI: the other changes are also present in #158 because i branched off of it! 👍🏼

@michelengelen
Copy link
Member Author

@prakhargupta1 do you have an idea why the Cirlce CI steps are not running?

@michelengelen michelengelen merged commit 2e8907e into mui:master May 6, 2024
2 checks passed
@michelengelen michelengelen deleted the automation/create-reusable-workflows branch May 6, 2024 10:28
@michelengelen michelengelen mentioned this pull request May 6, 2024
@prakhargupta1
Copy link
Member

@prakhargupta1 do you have an idea why the Cirlce CI steps are not running?

Nope, it did run in the Circle CI dashboard though. Your old PRs also seem to have the same concern.
cc: @mui/code-infra

DATA=$(gh api \
-H "Accept: application/vnd.github+json" \
-H "X-GitHub-Api-Version: 2022-11-28" \
/orgs/${{ github.repository_owner }}/teams/${{ inputs.team-slug }}/members?role=maintainer&per_page=100 \
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't understand this logic. Is this PR meant to experiment with reusable workflow?

@oliviertassinari
Copy link
Member

oliviertassinari commented May 10, 2024

do you have an idea why the Cirlce CI steps are not running?

@michelengelen #162 (comment). It should work now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: infra Org infrastructure work going on behind the scenes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants