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.

Sign up for GitHub

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

Jump to bottom

ansible-galaxy, az: update Italian translations #14513

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

ansible-galaxy, az: update Italian translations #14513

wants to merge 15 commits into from

Conversation

Astr0surf3r
Copy link
Contributor

fix 2 files part of the #101 issue for the IT translations

da-caricare

  • The page(s) are in the correct platform directories: common
  • The page(s) have at most 8 examples.
  • The page description(s) have links to documentation or a homepage.
  • The page(s) follow the content guidelines.
  • The page(s) follow the style guide.
  • The PR title conforms to the recommended templates.

@github-actions github-actions bot added the translation Translate pages from one language to another. label Oct 31, 2024
@tldr-bot

This comment has been minimized.

Sign in to view
@Astr0surf3r Astr0surf3r changed the title Outdate malformed pages pages.it / Outdate malformed pages Oct 31, 2024
@sebastiaanspeck sebastiaanspeck changed the title pages.it / Outdate malformed pages ansible-galaxy, az: update Italian translations Nov 1, 2024
@tldr-bot

This comment has been minimized.

Sign in to view
kbdharun
kbdharun approved these changes Nov 2, 2024
View reviewed changes
Copy link
Member

@kbdharun kbdharun left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks for your contribution.

@Astr0surf3r
Copy link
Contributor Author

LGTM, Thanks for your contribution.

thank you

gutjuri
gutjuri approved these changes Nov 4, 2024
View reviewed changes
pages.it/common/az.md Outdated Show resolved Hide resolved
@sebastiaanspeck
Copy link
Member

FYI; we are waiting for a review from one of the CODEOWNERS, as they are native Italian speakers. If we do not get a review within 10 days, we will still merge it.

mebeim
mebeim requested changes Nov 9, 2024
View reviewed changes
Copy link
Member

@mebeim mebeim left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution @Astr0surf3r. The az page looks nice, but I think there are a lot of minor translation issues with the ansible-galaxy page (see below). You can either apply those here or I can do it for you, just let me know. Thanks again!

pages.it/common/ansible-galaxy.md Show resolved Hide resolved
pages.it/common/ansible-galaxy.md Outdated Show resolved Hide resolved
pages.it/common/ansible-galaxy.md Outdated Show resolved Hide resolved
pages.it/common/ansible-galaxy.md Outdated Show resolved Hide resolved
pages.it/common/ansible-galaxy.md Outdated Show resolved Hide resolved
pages.it/common/ansible-galaxy.md Outdated Show resolved Hide resolved
pages.it/common/ansible-galaxy.md Outdated Show resolved Hide resolved
pages.it/common/ansible-galaxy.md Outdated Show resolved Hide resolved
pages.it/common/ansible-galaxy.md Outdated Show resolved Hide resolved
pages.it/common/az.md

`az interactive`

- Mostra le descrizioni dei comandi:
Copy link
Member

@mebeim mebeim Nov 9, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

This should technically be "Mostra l'aiuto" or something like that if we want to be faithful to the English version ("Show help"), but it is nonetheless clear so I think it's also ok to keep it as is. Just a suggestion, do as you wish.

Copy link
Contributor Author

@Astr0surf3r Astr0surf3r Nov 13, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

@mebeim thanks so much for your tweaks! Perhaps we need a guideline for the Italian translation of tldr maybe a gist? What do you think?"

Copy link
Member

Choose a reason for hiding this comment

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

You can write an own style guide specifically for Italian, just as is done for German, Korean and Chinese: https://github.com/tldr-pages/tldr/tree/main/contributing-guides

Copy link
Member

Choose a reason for hiding this comment

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

Good idea, though I am not sure I have the time now. But that's definitely something that can be done.

mebeim
mebeim requested changes Nov 14, 2024
View reviewed changes
Copy link
Member

@mebeim mebeim left a comment

Choose a reason for hiding this comment

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

Hey @Astr0surf3r thanks for the changes, but I think I have missed a few of them in the last review. There are still some instances where third person is used (see below). I can also apply these for you if you want, LMK.

pages.it/common/az.md
@@ -8,22 +8,30 @@

`az login`

- Gestisci la tua iscrizione ad Azure:
- Gestisce il tuo abbonamento ad Azure:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- Gestisce il tuo abbonamento ad Azure:
- Gestisci il tuo abbonamento ad Azure:

pages.it/common/az.md

`az vm list`

- Gestisci i servizi Azure Kubernetes:
- Gestisce i servizi Azure Kubernetes:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- Gestisce i servizi Azure Kubernetes:
- Gestisci i servizi Azure Kubernetes:

pages.it/common/az.md

`az aks`

- Gestisci le risorse della rete di Azure:
- Gestisce le risorse della rete di Azure:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- Gestisce le risorse della rete di Azure:
- Gestisci le risorse della rete di Azure:

pages.it/common/ansible-galaxy.md
@@ -1,32 +1,32 @@
# ansible-galaxy

> Crea e gestisci ruoli di Ansible.
> Esegui varie operazioni inerenti i Ruoli e le Collezioni in Ansible.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
> Esegui varie operazioni inerenti i Ruoli e le Collezioni in Ansible.
> Esegui varie operazioni inerenti ai Ruoli e alle Collezioni in Ansible.

pages.it/common/ansible-galaxy.md

- Crea un nuovo ruolo:
- Ottieni informazioni inerenti un ruolo:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- Ottieni informazioni inerenti un ruolo:
- Ottieni informazioni inerenti a un ruolo:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sebastiaanspeck sebastiaanspeck sebastiaanspeck approved these changes

@mebeim mebeim mebeim requested changes

@spageektti spageektti spageektti approved these changes

@karthik-script karthik-script karthik-script approved these changes

@kbdharun kbdharun kbdharun approved these changes

@gutjuri gutjuri gutjuri approved these changes

@tansiret tansiret Awaiting requested review from tansiret tansiret is a code owner

@Magrid0 Magrid0 Awaiting requested review from Magrid0 Magrid0 is a code owner

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
translation Translate pages from one language to another.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@Astr0surf3r @tldr-bot @sebastiaanspeck @mebeim @gutjuri @kbdharun @karthik-script @spageektti