mirror of
https://github.com/flutter/flutter.git
synced 2025-06-03 00:51:18 +00:00
Add the ios-reviewers review team (#166034)
## Pre-launch Checklist - [x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs. - [x] I read the [Tree Hygiene] wiki page, which explains my responsibilities. - [x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement]. - [x] I signed the [CLA]. - [ ] I listed at least one issue that this PR fixes in the description above. - [x] I updated/added relevant documentation (doc comments with `///`). - [ ] I added new tests to check the change I am making, or this PR is [test-exempt]. - [x] I followed the [breaking change policy] and added [Data Driven Fixes] where supported. - [x] All existing and new tests are passing. If you need help, consider asking for advice on the #hackers-new channel on [Discord]. <!-- Links --> [Contributor Guide]: https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#overview [Tree Hygiene]: https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md [test-exempt]: https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#tests [Flutter Style Guide]: https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md [Features we expect every widget to implement]: https://github.com/flutter/flutter/blob/main/docs/contributing/Style-guide-for-Flutter-repo.md#features-we-expect-every-widget-to-implement [CLA]: https://cla.developers.google.com/ [flutter/tests]: https://github.com/flutter/tests [breaking change policy]: https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#handling-breaking-changes [Discord]: https://github.com/flutter/flutter/blob/main/docs/contributing/Chat.md [Data Driven Fixes]: https://github.com/flutter/flutter/blob/main/docs/contributing/Data-driven-Fixes.md --------- Co-authored-by: Jenn Magder <magder@google.com>
This commit is contained in:
parent
751924dbc9
commit
8d49d67564
1
.github/labeler.yml
vendored
1
.github/labeler.yml
vendored
@ -229,6 +229,7 @@ team-infra:
|
||||
- any-glob-to-any-file:
|
||||
- docs/infra/**/*
|
||||
|
||||
# Keep this synced with CODEOWNERS.
|
||||
team-ios:
|
||||
- changed-files:
|
||||
- any-glob-to-any-file:
|
||||
|
@ -21,3 +21,12 @@
|
||||
# scrutinized as mistakes are irreversible.
|
||||
/engine/src/flutter/shell/platform/embedder/embedder.h @cbracken @chinmaygarde @loic-sharma
|
||||
/engine/src/flutter/shell/platform/embedder/tests/embedder_frozen.h @cbracken @chinmaygarde @loic-sharma
|
||||
|
||||
# iOS team - keep this synced with .github/labeler.yml's team-ios section.
|
||||
/engine/src/flutter/shell/platform/darwin/common/ @ios-reviewers
|
||||
/engine/src/flutter/shell/platform/darwin/ios/framework/ @ios-reviewers
|
||||
/packages/flutter_tools/**/ios/* @ios-reviewers
|
||||
/packages/flutter_tools/**/macos/* @ios-reviewers
|
||||
/packages/flutter_tools/**/*xcode* @ios-reviewers
|
||||
/packages/flutter_tools/**/*ios* @ios-reviewers
|
||||
/packages/flutter_tools/**/*macos* @ios-reviewers
|
||||
|
@ -24,7 +24,7 @@ Our code of conduct is much, much stricter than most. We do not wait until someo
|
||||
|
||||
When you see something that might be described as unwelcoming or is in some other way a violation of our code of conduct, promptly contact the offender and ask them to read the code of conduct and consider how they might more effectively espouse its philosophy. Most people react very positively to this.
|
||||
|
||||
If they react negatively, or if they continue to make the environment unpleasant, they should be removed from the environment. On Discord, this would be kicking them from the channel. Repeat offenders should be banned. On GitHub, they can be blocked from our organisation (you may need to ask @Hixie or another admin of our GitHub org to do this). Please let the #server-support [Chat](Chat.md) channel know when you do anything like this, so that we can keep an eye on how common it is.
|
||||
If they react negatively, or if they continue to make the environment unpleasant, they should be removed from the environment. On Discord, this would be kicking them from the channel. Repeat offenders should be banned. On GitHub, they can be blocked from our organization (you can ask @github-admin on Discord to do this). Please let the #server-support [Chat](Chat.md) channel know when you do anything like this, so that we can keep an eye on how common it is.
|
||||
|
||||
#### Maintaining documentation
|
||||
|
||||
@ -118,20 +118,21 @@ To add a fcontrib.org participant:
|
||||
|
||||
Some parts of the codebase have teams specified so that PRs get round-robin assigned for review.
|
||||
|
||||
To join one of these teams, request members be added/deleted, or change any settings, ping @Hixie on Discord. Members must be a member of the Flutter Hackers group (as documented at the top of this page).
|
||||
To join one of these teams, request members be added/deleted, or change any settings, ping @github-admin on Discord. Members must be a member of the Flutter Hackers group (as documented at the top of this page).
|
||||
|
||||
We currently have the following review teams:
|
||||
|
||||
* [`android-reviewers`](https://github.com/orgs/flutter/teams/android-reviewers): for folks working on the Android port of Flutter; use `#hackers-android` for discussions.
|
||||
* [`devtools-reviewers`](https://github.com/orgs/flutter/teams/devtools-reviewers): for the [devtools](https://github.com/flutter/devtools) repo; use `#hackers-devexp` for discussions.
|
||||
* [`ios-reviewers`](https://github.com/orgs/flutter/teams/ios-reviewers): for folks working on the iOS port of Flutter; use `#hackers-ios` for discussions.
|
||||
* [`website-reviewers`](https://github.com/orgs/flutter/teams/website-reviewers): for folks working on www.flutter.dev and docs.flutter.dev; use `#hackers-devrel` for discussions.
|
||||
|
||||
To create a new team, contact @Hixie. You will also need to create a `CODEOWNERS` file to actually trigger the review assignment.
|
||||
To create a new team, contact @github-admin on Discord. You will also need to create a `CODEOWNERS` file to actually trigger the review assignment.
|
||||
|
||||
# Pusher permissions
|
||||
|
||||
Some branches are protected to avoid accidents. Only people in the specific branches can push to them. Anyone can ask to be added or removed from these groups, they exist only to reduce accidents, not for security.
|
||||
|
||||
To join one of these teams, request members be added/deleted, or change any settings, ping @Hixie on Discord. Members must be a member of the Flutter Hackers group (as documented at the top of this page).
|
||||
To join one of these teams, request members be added/deleted, or change any settings, ping @github-admin on Discord. Members must be a member of the Flutter Hackers group (as documented at the top of this page).
|
||||
|
||||
The following groups have been defined for these purposes: pushers-beta, pushers-fuchsia
|
||||
The following groups have been defined for these purposes: pushers-beta, pushers-fuchsia
|
||||
|
Loading…
Reference in New Issue
Block a user