Skip to content

Conversation

@nfantoni
Copy link

@nfantoni nfantoni commented Oct 30, 2025

This pull request contain an italian translation for the topic section of the helm site. Let me know if something is missing or there are any errors. Thank you

This is the evolution of the pr #1602

@TerryHowe TerryHowe requested a review from Copilot November 7, 2025 15:14
Copy link
Contributor

@TerryHowe TerryHowe left a comment

Choose a reason for hiding this comment

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

Seems like this PR should not remove yarn.lock

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request adds Italian language support to the Helm documentation website. The changes include translations for navigation components, documentation pages, and configuration updates to enable Italian as a supported locale.

Key Changes

  • Added Italian locale configuration to Docusaurus
  • Translated navigation elements (navbar, footer)
  • Translated multiple documentation pages covering topics like charts, plugins, registries, RBAC, and version management
  • Added Italian translations for UI elements and common phrases

Reviewed Changes

Copilot reviewed 27 out of 28 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
docusaurus.config.js Added "it" to the locales array and configured Italian language label
i18n/it/docusaurus-theme-classic/*.json Translated navigation bar and footer text strings
i18n/it/docusaurus-plugin-content-docs/*.json Translated version labels and sidebar categories
i18n/it/docusaurus-plugin-content-docs/version-3/topics/*.md Translated documentation pages for various Helm topics
i18n/it/docusaurus-plugin-content-docs/version-3/*.mdx Translated main documentation index and topic guide pages
i18n/it/code.json Translated UI strings and messages for the website interface
i18n/it/docusaurus-plugin-content-blog/options.json Translated blog-related strings

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@nfantoni nfantoni requested a review from TerryHowe November 7, 2025 16:28
@nfantoni
Copy link
Author

nfantoni commented Nov 7, 2025

Seems like this PR should not remove yarn.lock

Hi @TerryHowe , I'm sorry. I have restored the file

Copy link
Contributor

@TerryHowe TerryHowe left a comment

Choose a reason for hiding this comment

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

/lgtm

Preview looks good to me.

Scott or Paige would know better if this meets all requirements of the new site.

Thanks for keeping this work moving!

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 27 out of 27 changed files in this pull request and generated 7 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@TerryHowe
Copy link
Contributor

Take a look at copilot's comments. Some of them look like good feedback.

Once that is resolved, hopefully I can round up another reviewer and push this through. Thanks.

@nfantoni
Copy link
Author

Take a look at copilot's comments. Some of them look like good feedback.

Once that is resolved, hopefully I can round up another reviewer and push this through. Thanks.

hi @TerryHowe ,

done the copilot suggestion. All the request was good!

Thanks

Copy link
Contributor

@TerryHowe TerryHowe left a comment

Choose a reason for hiding this comment

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

/lgtm

Hold this as it is unless there is a conflict, thanks!

Copy link
Contributor

@TerryHowe TerryHowe left a comment

Choose a reason for hiding this comment

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

/lgtm

Hold this as it is unless there is a conflict, thanks!

@nfantoni
Copy link
Author

/lgtm

Hold this as it is unless there is a conflict, thanks!

Hi @TerryHowe ,

so I don't have to merge the latest changes from the main branch unless there are conflicts to resolve?

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants