Remove inactive reviewers #14798
Merged
Remove inactive reviewers #14798
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
wpt-pr-bot
added
2dcontext
accelerometer
ambient-light
assets
battery-status
bluetooth
cookies
css-backgrounds
css-color
css-display
css-exclusions
css-flexbox
css-grid
css-logical
css-position
css-regions
css-scrollbars
css-shapes
css-style-attr
css-syntax
css-text
css-transforms
css-ui
css-writing-modes
CSS2
device-memory
docs
encrypted-media
event-timing
labels
|
I'm fine for adding me to css-scrollbars. |
|
I'm fine to be added as feature-policy reviewer. |
|
Thanks @bakulf. Suggested you https://github.com/web-platform-tests/wpt/pull/14798/files#r247437966 |
|
OK, so that leaves |
|
You can add me as the css-position reviewer for now. |
|
For encrypted-media: @ddorwin, @jrummell-chromium? @cpearce? @jyavenard? |
Could you swap @ddorwin with @joeyparrish ? |
birtles
approved these changes
Jan 21, 2019
alvestrand
approved these changes
Jan 21, 2019
This removes reviewers who have had more than three review requests in the past year but who haven't reviewed anything.
Co-Authored-By: gsnedders <me@gsnedders.com>
gsnedders
force-pushed
the
inactive-reviewers
branch
from
6924b40
to
0760547
Compare
marcoscaceres
pushed a commit
that referenced
this pull request
Jul 23, 2019
This removes reviewers who have had more than three review requests in the past year but who haven't reviewed anything. Adds a few extra reviewers based on discussions in the PR thread looking for reviewers for directories that otherwise would've ended up with none.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This removes reviewers who have had more than three review requests in the past year but who haven't reviewed anything. Fixes #13702.
Places where this leaves us with no reviewers:
css/css-position (very few PRs; @atanassov is the current reviewer)css/css-scrollbars (very few PRs; @tantek is current? maybe +@upsuper?)feature-policy (very few PRs; @clelland is current)gamepad (very few PRs; @luser is current)I believe everyone removed here was pinged in #13702 (comment), so I don't think we need to check about anyone except for the above. If the people mentioned above want to remain reviewers, please speak now.