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

Add Finnish translation #595

Open
wants to merge 95 commits into
base: master
Choose a base branch
from
Open

Conversation

korhox
Copy link

@korhox korhox commented Jul 20, 2024

Hello!

I've prepared with my friends @masisoranummi, @villesalin and @JesseSchemeikka a Finnish translation for the Conventional Commits specificaiton.

I was able to spin up the dev server locally, but I'd recommend checking the settings twice before pushing to production.

Let me know, if you guys need any help or information about the translations or if we've missed something, as the translation was made some time ago and something may have changed after that. I'm more than happy to make adjustments when required.

= and others added 30 commits October 13, 2022 12:51
feat(fi): merge masi's test
Add translations by Ville
feat(fi): translated first five lines of Specification
masisoranummi and others added 28 commits November 14, 2022 12:27
feat(fi): translate three more lines from Specification
feat(fi): translate last 3 lines of Specification
feat(fi): translate section: "Why Use Conventional Commits"
@korhox
Copy link
Author

korhox commented Jul 20, 2024

Also, I added the Finnish language with currently highest weight, as I was not sure about the language order. I'd prefer to languages be alphabetically ordered, but I didn't took liberty to reorder them.

@damianopetrungaro
Copy link
Member

Thank you very much @korhox, @masisoranummi, @villesalin and @JesseSchemeikka !
Love the team effort you have put here, may I ask you just to commit minimal changes to the PR? I can see a lot of formatting changes in the config.yaml that makes the code hard to review.

Please ping me once you are done, really happy to merge this ASAP!

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.

5 participants