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

[hi] Add content/hi/docs/tasks/tools/included/optional-kubectl-confi… #30627

Merged
merged 8 commits into from
Nov 28, 2021
Merged

[hi] Add content/hi/docs/tasks/tools/included/optional-kubectl-confi… #30627

merged 8 commits into from
Nov 28, 2021

Conversation

ashish-jaiswar
Copy link
Contributor

Added one file

localize optional-kubectl-configs-bash-linux.md to Hindi.
issue

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Nov 24, 2021
@k8s-ci-robot k8s-ci-robot added language/hi Issues or PRs related to Hindi language sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels Nov 24, 2021
@ashish-jaiswar
Copy link
Contributor Author

hey @anubha-v-ardhan can you review this PR ?

Comment on lines 3 to 9
description: "Linux पर bash ऑटो-कम्पलीशन के लिए कुछ वैकल्पिक विन्विन्यास।"
headless: true
---

## परिचय

Bash के लिए kubectl कम्प्लिशन स्क्रिप्ट `kubectl completion bash` कमांड के साथ उत्पन्न की जा सकती है। आपके शेल में कम्प्लिशन स्क्रिप्ट को सोर्स करने से kubectl ऑटोकम्प्लिशन ऐनेबल हो जाती है।
Copy link
Contributor

Choose a reason for hiding this comment

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

We'd want to use a fixed spelling for completion : here we have used कम्पलीशन and कम्प्लिशन both. I suggest going forward with कम्पलीशन

@ashish-jaiswar
Copy link
Contributor Author

@Garima-Negi, @anubha-v-ardhan I have updated the file, can you review this file?

Copy link
Contributor

@divya-mohan0209 divya-mohan0209 left a comment

Choose a reason for hiding this comment

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

PTAL


## परिचय

Bash के लिए kubectl कम्पलीशन स्क्रिप्ट `kubectl completion bash` कमांड के साथ उत्पन्न की जा सकती है। आपके शेल में कम्पलीशन स्क्रिप्ट को सोर्स करने से kubectl ऑटोकम्पलीशन ऐनेबल हो जाती है।
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe कम्पलीशन can be replaced by समापन? Or are we trying not to stray too far away from the terminology that is being introduced?

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 I replace कम्पलीशन by समापन, then what about this auto-completion, bash-completion ?


## परिचय

Bash के लिए kubectl कम्पलीशन स्क्रिप्ट `kubectl completion bash` कमांड के साथ उत्पन्न की जा सकती है। आपके शेल में कम्पलीशन स्क्रिप्ट को सोर्स करने से kubectl ऑटोकम्पलीशन ऐनेबल हो जाती है।
Copy link
Contributor

Choose a reason for hiding this comment

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

Also ऐनेबल can be replaced by सक्षम करना


## परिचय

Bash के लिए kubectl कम्पलीशन स्क्रिप्ट `kubectl completion bash` कमांड के साथ उत्पन्न की जा सकती है। आपके शेल में कम्पलीशन स्क्रिप्ट को सोर्स करने से kubectl ऑटोकम्पलीशन ऐनेबल हो जाती है।
Copy link
Contributor

Choose a reason for hiding this comment

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

Extra space between kubectl and कम्पलीशन


Bash के लिए kubectl कम्पलीशन स्क्रिप्ट `kubectl completion bash` कमांड के साथ उत्पन्न की जा सकती है। आपके शेल में कम्पलीशन स्क्रिप्ट को सोर्स करने से kubectl ऑटोकम्पलीशन ऐनेबल हो जाती है।

हालाँकि, कम्पलीशन की स्क्रिप्ट [**bash-completion**](https://github.com/scop/bash-completion) पर निर्भर करती हैं जिसका अर्थ है कि आपको पहले इस सॉफ़्टवेयर को स्थापित करना होगा (आप `type _init_completion` चलाकर परीक्षण कर सकते हैं कि आपने पहले से bash-completion इंस्टॉल की है या नहीं)।
Copy link
Contributor

Choose a reason for hiding this comment

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

हालाँकि, कम्पलीशन की स्क्रिप्ट bash-completion पर निर्भर करती हैं

Can be rephrased for grammatical correctness as below

हालाँकि, कम्पलीशन की स्क्रिप्ट bash-completion पर निर्भर हैं


Bash के लिए kubectl कम्पलीशन स्क्रिप्ट `kubectl completion bash` कमांड के साथ उत्पन्न की जा सकती है। आपके शेल में कम्पलीशन स्क्रिप्ट को सोर्स करने से kubectl ऑटोकम्पलीशन ऐनेबल हो जाती है।

हालाँकि, कम्पलीशन की स्क्रिप्ट [**bash-completion**](https://github.com/scop/bash-completion) पर निर्भर करती हैं जिसका अर्थ है कि आपको पहले इस सॉफ़्टवेयर को स्थापित करना होगा (आप `type _init_completion` चलाकर परीक्षण कर सकते हैं कि आपने पहले से bash-completion इंस्टॉल की है या नहीं)।
Copy link
Contributor

Choose a reason for hiding this comment

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

There are multiple double spaces in the second line. Please amend.


## Bash-completion को इंस्टॉल करें

कई पैकेज मैनेजर द्वारा bash-completion प्रदान की जाती है ([यहाँ](https://github.com/scop/bash-completion#installation) देखें)। आप इसे `apt-get install bash-completion` या `yum install bash-completion` आदि के साथ इंस्टॉल कर सकते हैं।
Copy link
Contributor

Choose a reason for hiding this comment

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

We need to use either इंस्टॉल or स्थापित. Using both of them will be unnecessary across the document. Please make amends.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok, I will update.

@ashish-jaiswar
Copy link
Contributor Author

@divya-mohan0209 , @Garima-Negi , @anubha-v-ardhan I have updated the file, Can I squash the commit?

@ashish-jaiswar
Copy link
Contributor Author

@divya-mohan0209 can you merge this PR? I have updated this file.

Copy link
Member

@anubha-v-ardhan anubha-v-ardhan left a comment

Choose a reason for hiding this comment

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

some NITS, LGTM otherwise

ashish-jaiswar and others added 5 commits November 28, 2021 23:10
…bash-linux.md

Co-authored-by: Anubhav Vardhan <vardhananubhav@gmail.com>
…bash-linux.md

Co-authored-by: Anubhav Vardhan <vardhananubhav@gmail.com>
…bash-linux.md

Co-authored-by: Anubhav Vardhan <vardhananubhav@gmail.com>
…bash-linux.md

Co-authored-by: Anubhav Vardhan <vardhananubhav@gmail.com>
…bash-linux.md

Co-authored-by: Anubhav Vardhan <vardhananubhav@gmail.com>
@anubha-v-ardhan
Copy link
Member

anubha-v-ardhan commented Nov 28, 2021

/label tide/merge-method-squash
/assign

@ashish-jaiswar If you know how to squash commits down to 1 commit, can you do it? Its okay if you don't too.

@k8s-ci-robot k8s-ci-robot added the tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. label Nov 28, 2021
@ashish-jaiswar
Copy link
Contributor Author

@anubha-v-ardhan I will squash the commit and merge it into a single commit.

@ashish-jaiswar
Copy link
Contributor Author

@anubha-v-ardhan can you squash the commit?

@anubha-v-ardhan
Copy link
Member

/lgtm
/approve

Thank you @ashish-jaiswar

@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: 1680635a8d2a79df14ec7bf170b1e071a398a8fe

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Nov 28, 2021
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: anubha-v-ardhan

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 Nov 28, 2021
@k8s-ci-robot k8s-ci-robot merged commit 76c2281 into kubernetes:dev-1.22-hi.1 Nov 28, 2021
@ashish-jaiswar ashish-jaiswar deleted the New_Branch branch November 28, 2021 18:50
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/hi Issues or PRs related to Hindi 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/M Denotes a PR that changes 30-99 lines, ignoring generated files. tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants