Skip to content

update i18n for Italian language#27

Open
valepert wants to merge 1 commit intowikimedia-gadgets:masterfrom
valepert:i18n-it-update
Open

update i18n for Italian language#27
valepert wants to merge 1 commit intowikimedia-gadgets:masterfrom
valepert:i18n-it-update

Conversation

@valepert
Copy link
Copy Markdown
Contributor

No description provided.

@Joeytje50
Copy link
Copy Markdown
Collaborator

Thanks for submitting some updates to the i18n file! I've got a request for you though; in PR #15, some changes were suggested to the i18n file as well. Since I don't speak any Italian, I was a bit hesitant to merge these changes without checking with someone else, and I never really put in the time to look for someone to check. Since you've also submitted the original i18n file, would you be willing to have a look at those changes as well, and let me know which of those changes look good?

@valepert
Copy link
Copy Markdown
Contributor Author

valepert commented May 3, 2025

I think @Mannivu PR fixes mainly indentation/tabulation problems, the only real change (L232) is "es." instead of "e.g." that is correct. For me it's ok to merge #15 before this one but I don't know if I need to rebase/edit my branch to accomodate the new it.js.

@Mannivu
Copy link
Copy Markdown

Mannivu commented May 3, 2025

I don't any problem to cancel my PR (I see that it has a conflict which I didn't see two years ago). My fix can be added to @valepert's PR since it was a minor fix.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants