-
Notifications
You must be signed in to change notification settings - Fork 10k
CODEOWNER and chore assignee changes #26379
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
base: production
Are you sure you want to change the base?
Conversation
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
|
CI run failed: build logs |
|
Preview URL: https://41d050f8.preview.developers.cloudflare.com |
| assignees.add(owner.replace(/^@/, "")); | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We'll need to build the corresponding .js files due to these changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We must keep this closing curly bracket (closes the if block started above in line 74).
| assignees.add(owner.replace(/^@/, "")); | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We must keep this closing curly bracket (closes the for loop started above in line 55).
Summary
Remove myself, Denise, Patricia and Pedro as auto-assignees to issues and PRs without dedicated owners in the CODEOWNERS file
Remove myself as auto-assignee for two chores