Skip to content
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

Update site config for l10n teams and 1.11 info #10667

Merged
merged 1 commit into from
Oct 23, 2018
Merged

Update site config for l10n teams and 1.11 info #10667

merged 1 commit into from
Oct 23, 2018

Conversation

zacharysarah
Copy link
Contributor

/ref #10666

This PR updates release-1.11 with:

@kubernetes-docs-i18n-bot kubernetes-docs-i18n-bot added the language/en Issues or PRs related to English language label Oct 18, 2018
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. do-not-merge/invalid-owners-file Indicates that a PR should not merge because it has an invalid OWNERS file in it. labels Oct 18, 2018
Copy link
Contributor

@k8s-ci-robot k8s-ci-robot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@zacharysarah: 4 invalid OWNERS files

In response to this:

/ref #10666

This PR updates release-1.11 with:

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/test-infra repository.

@@ -0,0 +1,8 @@
# This is the directory for English source content.
# Teams and members are visible at https://github.com/orgs/kubernetes/teams.
reviewers:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cannot parse file: error converting YAML to JSON: yaml: line 3: did not find expected .

@@ -0,0 +1,8 @@
# This is the localization project for Japanese.
# Teams and members are visible at https://github.com/orgs/kubernetes/teams.
reviewers:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cannot parse file: error converting YAML to JSON: yaml: line 3: did not find expected .

@@ -0,0 +1,8 @@
# This is the localization project for Chinese.
# Teams and members are visible at https://github.com/orgs/kubernetes/teams.
reviewers:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cannot parse file: error converting YAML to JSON: yaml: line 3: did not find expected .

@@ -0,0 +1,8 @@
# This is the localization project for Chinese.
# Teams and members are visible at https://github.com/orgs/kubernetes/teams.
reviewers:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cannot parse file: error converting YAML to JSON: yaml: line 3: did not find expected .

Copy link
Contributor

@k8s-ci-robot k8s-ci-robot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@zacharysarah: 4 invalid OWNERS files

In response to this:

/ref #10666

This PR updates release-1.11 with:

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/test-infra repository.

@@ -0,0 +1,8 @@
# This is the localization project for Japanese.
# Teams and members are visible at https://github.com/orgs/kubernetes/teams.
reviewers:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cannot parse file: error converting YAML to JSON: yaml: line 3: did not find expected .

@@ -0,0 +1,8 @@
# This is the localization project for Chinese.
# Teams and members are visible at https://github.com/orgs/kubernetes/teams.
reviewers:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cannot parse file: error converting YAML to JSON: yaml: line 3: did not find expected .

@@ -0,0 +1,8 @@
# This is the localization project for Chinese.
# Teams and members are visible at https://github.com/orgs/kubernetes/teams.
reviewers:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cannot parse file: error converting YAML to JSON: yaml: line 3: did not find expected .

@@ -0,0 +1,8 @@
# This is the directory for English source content.
# Teams and members are visible at https://github.com/orgs/kubernetes/teams.
reviewers:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cannot parse file: error converting YAML to JSON: yaml: line 3: did not find expected .

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/invalid-owners-file Indicates that a PR should not merge because it has an invalid OWNERS file in it. label Oct 18, 2018
@zacharysarah
Copy link
Contributor Author

@zparnold 👋 Just realized there was a missed step in the 1.12 release: updating the release-1.11:config.toml to deprecate release-1.11 and add info for release-1.12. This PR should take care of it, just surfacing for visibility.

/assign @tfogo
See also #10668

Copy link
Contributor

@xiangpengzhao xiangpengzhao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some concerns :)

@@ -102,11 +123,29 @@ description = "Production-Grade Container Orchestration"
languageName ="English"
# Weight used for sorting.
weight = 1
[languages.cn]

[languages.zh]
title = "Kubernetes"
description = "Production-Grade Container Orchestration"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this description use Chinese? Same concern for below languages.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the team would like to submit a PR for Chinese strings after this PR merges, that would be awesome. 👍

languageName ="Norsk"
weight = 4
contentDir = "content/no"

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missed languages.ja?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@xiangpengzhao The team hasn't indicated that they've met the minimum milestones yet. @cstoku Any updates on progress?


[languages.no]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't see content/no/OWNERS file for this.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@xiangpengzhao It's not strictly needed, as content/no/ is currently a test directory.

@zacharysarah
Copy link
Contributor Author

zacharysarah commented Oct 19, 2018

@chenopis 👋 It looks like netlify is failing:

11:47:11 AM: ERROR 2018/10/19 18:47:11 error processing shortcode 
"shortcodes/deprecationwarning.html" for page "_index.html": template: 
shortcodes/deprecationwarning.html:7:65: executing "shortcodes/deprecationwarning.html" at 
<.Params.currenturl>: can't evaluate field currenturl in type interface {}

The value of currentURL matches that of release-1.12 and release-1.10 (#10688), which run without failing. I'm at a bit of a loss as to how to diagnose/resolve the cause here.

Any advice?

@zacharysarah
Copy link
Contributor Author

zacharysarah commented Oct 22, 2018

@chenopis Ping

UPDATE: Never mind, I figured it out. 👍

Update OWNERS_ALIASES to be current

Fix OWNERS files syntax

Fix currentUrl

Fix currentUrl even more

Standardize value of currentUrl

Fix layout shortcode syntax
config.toml Show resolved Hide resolved
@tfogo
Copy link
Contributor

tfogo commented Oct 23, 2018

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 23, 2018
@Bradamant3
Copy link
Contributor

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Bradamant3, tfogo

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 23, 2018
@k8s-ci-robot k8s-ci-robot merged commit d8d4111 into kubernetes:release-1.11 Oct 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/en Issues or PRs related to English language lgtm "Looks good to me", indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants