Skip to content

Commit

Permalink
Remove rsimha from all owners files and comments (ampproject#38142)
Browse files Browse the repository at this point in the history
  • Loading branch information
rsimha authored Apr 26, 2022
1 parent 9dcb735 commit b21e311
Show file tree
Hide file tree
Showing 13 changed files with 11 additions and 21 deletions.
2 changes: 1 addition & 1 deletion .circleci/OWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
{
rules: [
{
owners: [{name: 'ampproject/wg-infra'}, {name: 'rsimha', notify: true}],
owners: [{name: 'ampproject/wg-infra'}],
},
],
}
2 changes: 1 addition & 1 deletion .circleci/install_microsoft_edge.sh
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
#
# Script used by AMP's CI builds to install Microsoft Edge Beta on CircleCI.
# Reference: https://www.microsoftedgeinsider.com/en-us/download?platform=linux-deb
# TODO(rsimha): Switch from Beta to Stable once it's available.
# TODO(wg-infra): Switch from Beta to Stable once it's available.

set -e

Expand Down
2 changes: 1 addition & 1 deletion .vscode/OWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
{
rules: [
{
owners: [{name: 'ampproject/wg-infra'}, {name: 'rsimha', notify: true}],
owners: [{name: 'ampproject/wg-infra'}],
},
],
}
1 change: 0 additions & 1 deletion OWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@
{name: 'cramforce', requestReviews: false},
{name: 'dvoytenko', requestReviews: false},
{name: 'jridgewell', requestReviews: false},
{name: 'rsimha', requestReviews: false},
],
},
{
Expand Down
2 changes: 1 addition & 1 deletion build-system/common/OWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
{
rules: [
{
owners: [{name: 'ampproject/wg-infra'}, {name: 'rsimha', notify: true}],
owners: [{name: 'ampproject/wg-infra'}],
},
],
}
2 changes: 1 addition & 1 deletion build-system/common/ci.js
Original file line number Diff line number Diff line change
Expand Up @@ -172,7 +172,7 @@ function ciJobId() {
*/
function ciJobUrl() {
return isGithubActions
? // TODO(rsimha): Try to reverse engineer the GH Actions job URL from the build URL.
? // TODO(wg-infra): Try to reverse engineer the GH Actions job URL from the build URL.
`${env('GITHUB_SERVER_URL')}/${env('GITHUB_REPOSITORY')}/actions/runs/${env('GITHUB_RUN_ID')}` // prettier-ignore
: isCircleci
? env('CIRCLE_BUILD_URL')
Expand Down
2 changes: 1 addition & 1 deletion build-system/common/update-packages.js
Original file line number Diff line number Diff line change
Expand Up @@ -190,7 +190,7 @@ function patchPreact() {

/**
* Deletes the map file for rrule, which breaks closure compiler.
* TODO(rsimha): Remove this workaround after a fix is merged for
* TODO(wg-infra): Remove this workaround after a fix is merged for
* https://github.com/google/closure-compiler/issues/3720.
*/
function removeRruleSourcemap() {
Expand Down
2 changes: 1 addition & 1 deletion build-system/pr-check/OWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
{
rules: [
{
owners: [{name: 'ampproject/wg-infra'}, {name: 'rsimha', notify: true}],
owners: [{name: 'ampproject/wg-infra'}],
},
],
}
4 changes: 0 additions & 4 deletions build-system/server/OWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,5 @@
{
owners: [{name: 'ampproject/wg-infra'}],
},
{
pattern: 'lazy-build.js',
owners: [{name: 'rsimha', notify: true}],
},
],
}
2 changes: 1 addition & 1 deletion build-system/task-runner/OWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
{
rules: [
{
owners: [{name: 'ampproject/wg-infra'}, {name: 'rsimha', notify: true}],
owners: [{name: 'ampproject/wg-infra'}],
},
],
}
2 changes: 1 addition & 1 deletion build-system/tasks/runtime-test/custom-launchers.js
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ const customLaunchers = {
flags: argv.headless ? ['-headless'] : [],
},
EdgeCustom: {
// TODO(rsimha): Switch from Beta to Stable once it's available.
// TODO(wg-infra): Switch from Beta to Stable once it's available.
base: argv.headless ? 'EdgeBetaHeadless' : 'EdgeBeta',
flags: ['--no-sandbox', '--disable-gpu', '--disable-dev-shm-usage'],
},
Expand Down
2 changes: 1 addition & 1 deletion testing/console-logging-setup.js
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ export function setTestName(name) {

/**
* Exposes the test runner object so that errors can be associated with it.
* TODO(rsimha): Remove this after karma-runner/karma-mocha#236 is fixed.
* TODO(wg-infra): Remove this after karma-runner/karma-mocha#236 is fixed.
* @param {Object} runner
*/
export function setTestRunner(runner) {
Expand Down
7 changes: 1 addition & 6 deletions third_party/OWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,7 @@
{
rules: [
{
owners: [
{name: 'rsimha'},
{name: 'erwinmombay'},
{name: 'choumx'},
{name: 'jridgewell'},
],
owners: [{name: 'erwinmombay'}, {name: 'choumx'}, {name: 'jridgewell'}],
},
],
}

0 comments on commit b21e311

Please sign in to comment.