InnerSourceCommons / InnerSourcePatterns

Proven approaches that can guide you through applying open source best practices within your organization
https://patterns.innersourcecommons.org
Creative Commons Attribution Share Alike 4.0 International
739 stars 181 forks source link

Add new translation language: Galician #613

Closed psanxiao closed 10 months ago

psanxiao commented 10 months ago

This PR adds a new entirely translation to Galician language. Includes:

This translation was made possible thank you to Xunta de Galicia, local goverment of Galicia region, as part as project to foster InnerSource practiques among the Galician companies.

People who participate in the translation work were:

We hope you find interesting this contribution and we looking forward to your feedback. Let us know any change that we should made so that this PR could be integrated.

Thank you in advance for your time.

welcome[bot] commented 10 months ago

Thank You Banner

:sparkling_heart: Thanks for opening this pull request! :sparkling_heart: The InnerSource Commons community really appreciates your time and effort to contribute to the project. Please make sure you have read our Contributing Guidelines.

If you are submitting a new pattern, the following things will help get your pull request across the finish line! :checkered_flag:

This project has a small number of maintainers, volunteering their time to this project. So please be patient and we will get back to you as soon as we can. If we don't acknowledge this pull request after 7 days, feel free to chat to us about it in our Slack workspace.

spier commented 10 months ago

Awesomeness in the making @psanxiao! Will get back to this shortly.

spier commented 10 months ago

@psanxiao thank you again for this contribution.

I made various tiny fixes to the markdown formatting. Those were just house-keeping items though, that make it simpler for us to maintain these patterns.

Further I found some broken links, which I fixed as well.

Just for two of these links I wasn't sure. I found these links to files that don't exist:

I have fixed those links as well to point to the English version of those files instead. If you were still intending to translate those files to Galician too, you can certainly do that. However you don't have to as the translations to Japanese/Chinese etc have not done that either (i.e. they kept the English versions as well).

Also at least for the mind-map it will be a bit trickier to do the translation, as we have some automation there that generates the mindmap, which we would have to fix too.

psanxiao commented 10 months ago

Thank you very much @spier for reviewing and for all the fixes.

About those links that you mentioned, yeah, it was my fault. I did not upload those files in the previous commits. But, taking into account what you say, maybe it has no sense to upload the translation for those files.

spier commented 10 months ago

Oh, well if you already have those fixes anyways, by all means, please add them to the PR @psanxiao !

spier commented 10 months ago

Btw there is something that I need to clarify with the gitbook Support team before publishing the Galician version of the patterns. So it will likely take until early next week. Just to set expectations :)

psanxiao commented 10 months ago

Btw there is something that I need to clarify with the gitbook Support team before publishing the Galician version of the patterns. So it will likely take until early next week. Just to set expectations :)

No problem with that. Thank you very much for all your work done.

spier commented 10 months ago

Btw there is something that I need to clarify with the gitbook Support team before publishing the Galician version of the patterns. So it will likely take until early next week. Just to set expectations :)

No problem with that. Thank you very much for all your work done.

You got it :)

Actually I think I was able to resolve the issue. Should be able to release a bit sooner then :)

spier commented 10 months ago

Note to self: We will have to re-work the generate-mindmap.yml GitHub Action, to make sure that we re-generate the mindmap for Galician as well, if the input files change.

spier commented 10 months ago

Also after this PR we get to do some continuity testing immediate, as in the meantime there is another pattern that needs translation :) (see Standard Release Process)

spier commented 10 months ago

I think I checked this PR sufficiently now, and am happy to get this merged tomorrow, and published in our online book. This will be so cool 🥳

One more question for you though @psanxiao: For each translation we have a translation lead. We have established that concept, as we don't speak these languages ourselves, so naturally we cannot review future contributions to these translations ourselves.

So, would you like to be the translation lead for Galician? :)

psanxiao commented 10 months ago

So, would you like to be the translation lead for Galician? :)

Sure! :)

welcome[bot] commented 10 months ago

Congratulations Banner Congrats on merging your first pull request! :tada: We here at The InnerSource Commons are proud of you! :sparkling_heart: Thank you so much for your contribution :gift:

spier commented 10 months ago

I think I already said it more than once, but it doesn't hurt: Thank you again for all your work @psanxiao !!! ❤️

This is now live :)