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

i18n: update the Italian translation #90

Merged

Conversation

AlessandroMenti
Copy link
Contributor

This PR updates the Italian translation of this plugin.

Note that I have redone the translation from scratch; for this reason, the copyright notice in src/main/resources/hudson/tasks/Mailer/UserProperty/config_it.properties has been changed to include only my name.

Copy link
Member

@alecharp alecharp left a comment

Choose a reason for hiding this comment

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

Could you please review the modifications I suggested? Thanks.

@AlessandroMenti
Copy link
Contributor Author

No problem (in fact, I had also planned to convert all the files using native2ascii because, even though they are valid ISO 8859-1, if somebody edits them - e.g. to update the translated strings - they might inadvertently switch the encoding to UTF-8...)

@AlessandroMenti
Copy link
Contributor Author

@alecharp could you please review my modifications again?

@alecharp
Copy link
Member

Closing to re-trigger a build

@alecharp alecharp closed this Sep 28, 2020
@alecharp alecharp reopened this Sep 28, 2020
@alecharp
Copy link
Member

I believe the build issue is not related to the changeset of this PR. Closing to re-trigger the build.

@alecharp alecharp closed this Sep 29, 2020
@alecharp alecharp reopened this Sep 29, 2020
Copy link
Member

@oleg-nenashev oleg-nenashev left a comment

Choose a reason for hiding this comment

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

LGTM. CC @MarkEWaite for the localization content review

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.

3 participants