-
-
Notifications
You must be signed in to change notification settings - Fork 158
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 German Translation of 2025 GenAI Top 10 #528
base: translations/de-DE
Are you sure you want to change the base?
Conversation
|
2_0_vulns/translations/de-DE/baseline/custom_data_LLM_de-DE.json
Outdated
Show resolved
Hide resolved
2_0_vulns/translations/de-DE/baseline/custom_data_LLM_de-DE.json
Outdated
Show resolved
Hide resolved
Please squash the commits to one commit before merge. |
2_0_vulns/translations/de-DE/baseline/custom_data_LLM_de-DE.json
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Three check points:
- paragraph spacing
- "#" or "Nr" after "Szenario"
- NB_SPACE usage after ###$
Not all instances are commented; pls be comprehensive.
2_0_vulns/translations/de-DE/LLM08_VectorAndEmbeddingWeaknesses.md
Outdated
Show resolved
Hide resolved
2_0_vulns/translations/de-DE/baseline/custom_data_LLM_de-DE.json
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My last two commennts:
- 2025-01-13 for German release date
- Unnecessary new line in LLM06.
Otherwise, LGTM.
FYI -- How to remove appendices
or |
dce879c
to
a474a46
Compare
833ee54
to
d152cde
Compare
Thank you for your Feedback, @Setotet |
PR Checklist 🚨
Proposed changes
this PR will add a German translation of the 2025 GenAI Top 10.
A special thank you to @RicoKomenda for the excellent collaboration on this effort.
Types of changes
Checklist
Further comments
To ensure correct rendering, we verified the Markdown files using the OWASP PDF Tool 4.4.0 with the following command:
./owasp_pdf -y -l LLM_de-DE
Special thanks to @Setotet for his support.
Label Request
We kindly request that the
translation
label be added to this pull request. Unfortunately, we cannot apply it ourselves due to missing write access on the repository.