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 previously missing german translations #4473

Merged

Conversation

CuddlyBunion341
Copy link
Contributor

@CuddlyBunion341 CuddlyBunion341 commented Feb 20, 2024

Many translations needed to be included previously on rubygems.org, especially on the landing page. This Pull Request is my attempt at translating the locales based on English and French translations, despite https://github.com/rubygems/rubygems.org/blob/master/CONTRIBUTING.md#acceptance mentioning that the community won't accept contributions without tests. In my understanding, this rule shouldn't affect configuration/locale files.

Thank you for considering this contribution!

CleanShot 2024-02-20 at 15 30 39@2x

@simi
Copy link
Member

simi commented Feb 22, 2024

Hello @CuddlyBunion341 and thanks for the translations. I'll leave this open for few days to make it possible to another 🇩🇪 speaking person to do review. Per my basic 🇩🇪 language knowledge it looks ok.

No worries, tests are not needed for translations (should we update the template?).

Copy link

codecov bot commented Feb 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.15%. Comparing base (d0777b3) to head (398b148).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4473   +/-   ##
=======================================
  Coverage   97.15%   97.15%           
=======================================
  Files         391      391           
  Lines        8260     8260           
=======================================
  Hits         8025     8025           
  Misses        235      235           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@coorasse
Copy link
Contributor

@CuddlyBunion341 might you involve somebody else at Renuo for a review? We have tons of german speakers ;)

@CuddlyBunion341
Copy link
Contributor Author

I'll try my best to @coorasse.

Copy link

@ekyburz ekyburz left a comment

Choose a reason for hiding this comment

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

The german grammar looks great and I couldn't find any spelling mistakes.

@simi simi force-pushed the cuddlybunion341/add-german-translations branch from a0931e8 to 398b148 Compare February 25, 2024 19:32
@simi simi marked this pull request as ready for review February 25, 2024 19:32
@simi simi enabled auto-merge (squash) February 25, 2024 19:33
@simi simi merged commit 926b8b5 into rubygems:master Feb 25, 2024
17 checks passed
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.

None yet

4 participants