chore: add i18n OWNERS and OWNER_ALIASES for EN#748
Conversation
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: TineoC The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
k8s-ci-robot
added
cncf-cla: yes
TineoC
force-pushed
the
chore/governance-website-owners
branch
from
9b5733e to
dd79bd9
Compare
k8s-ci-robot
added
size/M
TineoC
changed the title
TineoC
force-pushed
the
chore/governance-website-owners
branch
2 times, most recently
from
8241a1a to
c1ca567
Compare
k8s-ci-robot
left a comment
k8s-ci-robot
left a comment
There was a problem hiding this comment.
@TineoC: 1 invalid OWNERS file
Details
In response to this:
Summary
Follow-up to #711 to split out the governance changes for website ownership and reviewer delegation.
Per review feedback on the i18n foundation PR, this splits the
sig-contribex-website-ownersandsig-contribex-website-reviewersaliases into their own PR so the governance change can be reviewed independently.Notes
This implements the reviewer suggestion to have at least two owners and to split the governance change out of the i18n foundation PR.
Fixes: #711 (follow-up)
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.
| # files/directories related to static site generator | ||
| "^(assets|layouts|static|package-lock\\.json|package\\.json|netlify\\.toml|hugo\\.yaml)$": | ||
| # files/directories related to static site generator and localization framework | ||
| "^(assets|layouts|static|package-lock\.json|package\.json|netlify\.toml|hugo\.yaml|content/)$": |
There was a problem hiding this comment.
Cannot parse file: error converting YAML to JSON: yaml: line 18: found unknown escape character.
TineoC
force-pushed
the
chore/governance-website-owners
branch
from
c1ca567 to
7fbe173
Compare
Add sig-contribex-website-owners and sig-contribex-website-reviewers aliases to OWNERS_ALIASES, and update the OWNERS content filter to use the website reviewers alias. - sig-contribex-website-owners: SIG ContribEx chairs and tech leads (Priyankasaggu11929, MadhavJivrajani, mfahlandt, palnabarun) - sig-contribex-website-reviewers: lmktfy, jberkus, reylejano - sig-contribex-en-owners: fsmunoz, TineoC, jberkus - sig-contribex-en-reviewers: lmktfy - OWNERS: add content/ to the static site generator filter with approvers Follow-up to the i18n foundation PR to split governance changes out.
TineoC
force-pushed
the
chore/governance-website-owners
branch
from
7fbe173 to
47065bb
Compare
k8s-ci-robot
left a comment
k8s-ci-robot
left a comment
There was a problem hiding this comment.
@TineoC: 1 invalid OWNERS file
Details
In response to this:
Summary
Follow-up to #711 to split out the governance changes for website ownership and reviewer delegation.
Per review feedback on the i18n foundation PR, this splits the
sig-contribex-website-ownersandsig-contribex-website-reviewersaliases into their own PR so the governance change can be reviewed independently.Notes
This implements the reviewer suggestion to have at least two owners and to split the governance change out of the i18n foundation PR.
Fixes: #711 (follow-up)
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.
| # files/directories related to static site generator | ||
| "^(assets|layouts|static|package-lock\\.json|package\\.json|netlify\\.toml|hugo\\.yaml)$": | ||
| # files/directories related to static site generator and localization framework | ||
| "^(assets|layouts|static|package-lock\.json|package\.json|netlify\.toml|hugo\.yaml|content/)$": |
There was a problem hiding this comment.
Cannot parse file: error converting YAML to JSON: yaml: line 18: found unknown escape character.
k8s-ci-robot
left a comment
k8s-ci-robot
left a comment
There was a problem hiding this comment.
@TineoC: 1 invalid OWNERS file
Details
In response to this:
Summary
Follow-up to #711 to split out the governance changes for website ownership and reviewer delegation.
Per review feedback on the i18n foundation PR, this splits the
sig-contribex-website-ownersandsig-contribex-website-reviewersaliases into their own PR so the governance change can be reviewed independently.Notes
This implements the reviewer suggestion to have at least two owners and to split the governance change out of the i18n foundation PR.
Fixes: #711 (follow-up)
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.
| # files/directories related to static site generator | ||
| "^(assets|layouts|static|package-lock\\.json|package\\.json|netlify\\.toml|hugo\\.yaml)$": | ||
| # files/directories related to static site generator and localization framework | ||
| "^(assets|layouts|static|package-lock\.json|package\.json|netlify\.toml|hugo\.yaml|content/)$": |
There was a problem hiding this comment.
Cannot parse file: error converting YAML to JSON: yaml: line 18: found unknown escape character.
k8s-ci-robot
removed
the
do-not-merge/invalid-owners-file
2 participants
Governance changes for website ownership and reviewer delegation, split out from the i18n foundation PR for independent review. Follow-up to #711.
Depends on: #711