Skip to content

[zh-cn]update configure-pod-configmap.md - #35411

Merged
k8s-ci-robot merged 1 commit into
kubernetes:mainfrom
yanrongshi:update-configure-pod-configmap.md
Aug 3, 2022
Merged

k8s-ci-robot merged 1 commit into
kubernetes:mainfrom
yanrongshi:update-configure-pod-configmap.md

Conversation

@yanrongshi

@yanrongshi yanrongshi commented Jul 26, 2022 •

Copy link
Copy Markdown
Contributor

@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. language/zh Issues or PRs related to Chinese language labels Jul 26, 2022
@k8s-ci-robot
k8s-ci-robot requested review from chenxuc and ydFu July 26, 2022 16:16
@k8s-ci-robot k8s-ci-robot added the sig/docs Categorizes an issue or PR as relevant to SIG Docs. label Jul 26, 2022
@netlify

netlify Bot commented Jul 26, 2022 •

Copy link
Copy Markdown

✅ Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit 6954524
🔍 Latest deploy log https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/62ea91847c276d00082a7d1b
😎 Deploy Preview https://deploy-preview-35411--kubernetes-io-main-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Comment thread content/zh-cn/docs/tasks/configure-pod-container/configure-pod-configmap.md Outdated
Comment thread content/zh-cn/docs/tasks/configure-pod-container/configure-pod-configmap.md Outdated
Comment thread content/zh-cn/docs/tasks/configure-pod-container/configure-pod-configmap.md Outdated
Comment thread content/zh-cn/docs/tasks/configure-pod-container/configure-pod-configmap.md Outdated
Comment thread content/zh-cn/docs/tasks/configure-pod-container/configure-pod-configmap.md Outdated
Comment thread content/zh-cn/docs/tasks/configure-pod-container/configure-pod-configmap.md Outdated
Comment thread content/zh-cn/docs/tasks/configure-pod-container/configure-pod-configmap.md Outdated
Comment thread content/zh-cn/docs/tasks/configure-pod-container/configure-pod-configmap.md Outdated
Comment thread content/zh-cn/docs/tasks/configure-pod-container/configure-pod-configmap.md Outdated
@yanrongshi
yanrongshi force-pushed the update-configure-pod-configmap.md branch 2 times, most recently from 7686dcc to 70df19b Compare July 27, 2022 09:01
@yanrongshi

Copy link
Copy Markdown
Contributor Author

@tengqm Thank you very much for checking it out, I have learned a lot from your comments

Comment thread content/zh-cn/docs/tasks/configure-pod-container/configure-pod-configmap.md Outdated
Comment thread content/zh-cn/docs/tasks/configure-pod-container/configure-pod-configmap.md Outdated
Comment thread content/zh-cn/docs/tasks/configure-pod-container/configure-pod-configmap.md Outdated
Comment thread content/zh-cn/docs/tasks/configure-pod-container/configure-pod-configmap.md Outdated
Comment thread content/zh-cn/docs/tasks/configure-pod-container/configure-pod-configmap.md Outdated
Comment thread content/zh-cn/docs/tasks/configure-pod-container/configure-pod-configmap.md Outdated
@yanrongshi
yanrongshi force-pushed the update-configure-pod-configmap.md branch from 70df19b to 197a960 Compare July 27, 2022 13:46
@yanrongshi
yanrongshi requested a review from tengqm July 27, 2022 13:47
@tengqm

tengqm commented Jul 27, 2022

Copy link
Copy Markdown
Contributor

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jul 27, 2022
@k8s-ci-robot

Copy link
Copy Markdown
Contributor

LGTM label has been added.

DetailsGit tree hash: 09d56b7faba09d66317bef70e637c851941f8268

Comment thread content/zh-cn/docs/tasks/configure-pod-container/configure-pod-configmap.md Outdated
Comment thread content/zh-cn/docs/tasks/configure-pod-container/configure-pod-configmap.md Outdated
Comment thread content/zh-cn/docs/tasks/configure-pod-container/configure-pod-configmap.md Outdated
Comment thread content/zh-cn/docs/tasks/configure-pod-container/configure-pod-configmap.md Outdated
@yanrongshi
yanrongshi force-pushed the update-configure-pod-configmap.md branch from 197a960 to 6954524 Compare August 3, 2022 15:17
@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 3, 2022
@yanrongshi

Copy link
Copy Markdown
Contributor Author

@Sea-n 感谢您细致的审查

@Sea-n

Sea-n commented Aug 3, 2022

Copy link
Copy Markdown
Member

Now this PR is good to be merged, thanks for your contribution! 🎉
/approve

And apply LGTM from previous review.
/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 3, 2022
@k8s-ci-robot

Copy link
Copy Markdown
Contributor

LGTM label has been added.

DetailsGit tree hash: 5d8613618eb8978f35652e3b143979eb990a1768

@k8s-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Sea-n

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

The pull request process is described here

Details 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 Aug 3, 2022
@k8s-ci-robot
k8s-ci-robot merged commit 3eced5d into kubernetes:main Aug 3, 2022
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/zh Issues or PRs related to Chinese language lgtm "Looks good to me", indicates that a PR is ready to be merged. sig/docs Categorizes an issue or PR as relevant to SIG Docs. 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.

4 participants