Skip to content

i18n(de): Project structure, German translation: fix grammar #12813

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.

Sign up for GitHub

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

Jump to bottom
Open
urban-adeininger wants to merge 1 commit into
base: main
Choose a base branch
from
Open

i18n(de): Project structure, German translation: fix grammar #12813

urban-adeininger wants to merge 1 commit into from
+1 −1

Conversation

@urban-adeininger
Copy link

@urban-adeininger urban-adeininger commented Dec 1, 2025 •
edited
Loading

This PR fixes the grammar in the German translation of the docs explaining the project structure.

@netlify
Copy link

netlify bot commented Dec 1, 2025 •
edited
Loading

✅ Deploy Preview for astro-docs-2 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 21f0ac9
🔍 Latest deploy log https://app.netlify.com/projects/astro-docs-2/deploys/692d84a66924070008cc2a08
😎 Deploy Preview https://deploy-preview-12813--astro-docs-2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions bot added the i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help! label Dec 1, 2025
@astrobot-houston
Copy link
Contributor

astrobot-houston commented Dec 1, 2025

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
de/basics/project-structure.mdx Localization changed, will be marked as complete.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@astrobot-houston
Copy link
Contributor

astrobot-houston commented Dec 1, 2025

Hello! Thank you for opening your first PR to Astro’s Docs! 🎉

Here’s what will happen next:

  1. Our GitHub bots will run to check your changes.
    If they spot any broken links you will see some error messages on this PR.
    Don’t hesitate to ask any questions if you’re not sure what these mean!

  2. In a few minutes, you’ll be able to see a preview of your changes on Netlify 🥳.

  3. One or more of our maintainers will take a look and may ask you to make changes.
    We try to be responsive, but don’t worry if this takes a few days.

@sarah11918 sarah11918 changed the title docs: Project structure, German translation: fix grammar i18n(de): Project structure, German translation: fix grammar Dec 1, 2025
randomguy-2650
randomguy-2650 approved these changes Dec 4, 2025
View reviewed changes
Copy link
Contributor

@randomguy-2650 randomguy-2650 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM! 🎉🧑‍🚀

Ich habe einige Kommentare über die kleinen Inkonsistenzen in dieser Datei geschrieben, die diese PR nicht ändert.

LGTM! 🎉🧑‍🚀

I left a few comments on just little inconsistencies in this file that this PR doesn’t change.

src/content/docs/de/basics/project-structure.mdx

Astro verarbeitet, optimiert und bündelt deine Dateien in `src/`, um die endgültige Website zu erstellen, die an den Browser gesendet wird. Im Gegensatz zum statischen `public/`-Verzeichnis werden deine `src/`-Dateien von Astro für dich verarbeitet und optimiert:

Einige Dateien (z.B. Astro-Komponenten) werden gar nicht so an den Browser gesendet, wie du sie geschrieben hast, sondern stattdessen in statisches HTML gerendert. Andere Dateien (z.B. CSS) werden zwar an den Browser gesendet, können aber zur Leistungsverbesserung vorher optimiert oder mit anderen CSS-Dateien gebündelt werden.
Copy link
Contributor

@randomguy-2650 randomguy-2650 Dec 4, 2025

Choose a reason for hiding this comment

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

Suggested change
Einige Dateien (z.B. Astro-Komponenten) werden gar nicht so an den Browser gesendet, wie du sie geschrieben hast, sondern stattdessen in statisches HTML gerendert. Andere Dateien (z.B. CSS) werden zwar an den Browser gesendet, können aber zur Leistungsverbesserung vorher optimiert oder mit anderen CSS-Dateien gebündelt werden.
Einige Dateien (z. B. Astro-Komponenten) werden gar nicht so an den Browser gesendet, wie du sie geschrieben hast, sondern stattdessen in statisches HTML gerendert. Andere Dateien (z. B. CSS) werden zwar an den Browser gesendet, können aber zur Leistungsverbesserung vorher optimiert oder mit anderen CSS-Dateien gebündelt werden.

Although this is not part of the PR, the i18n guide wants to use the correct abbreviation.

src/content/docs/de/basics/project-structure.mdx

Astro nutzt einige fest vorgegebene Verzeichnisse, um dein Projekt zu strukturieren. Das Stammverzeichnis jedes Astro-Projekts sollte die folgenden Verzeichnisse und Dateien enthalten:

- `src/*` - Der Quellcode deines Projekts (Komponenten, Seiten, Stile, Bilder, usw.)
Copy link
Contributor

@randomguy-2650 randomguy-2650 Dec 4, 2025

Choose a reason for hiding this comment

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

Suggested change
- `src/*` - Der Quellcode deines Projekts (Komponenten, Seiten, Stile, Bilder usw.)

I know this is unrelated.

Copy link
Contributor

@trueberryless trueberryless Dec 6, 2025

Choose a reason for hiding this comment

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

Git diff is not working here, did you just change the space between "Bilder" and "usw." to non-breaking unicode? If so, I approve ✅

Copy link
Contributor

@randomguy-2650 randomguy-2650 Dec 6, 2025

Choose a reason for hiding this comment

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

Yup, I did do that.

And GitHub is just plain annoying sometimes, it doesn’t even render the supposed “old code line in the proposal”.

Copy link
Contributor

@trueberryless trueberryless Dec 6, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

Yeah, I think it's currently very buggy when the suggestion is more than 2 lines before/after some edited lines... I hope GitHub will be able to stabilize this when the whole files tab will be rolled out of beta...

I reported this bug here.

trueberryless
trueberryless approved these changes Dec 6, 2025
View reviewed changes
Copy link
Contributor

@trueberryless trueberryless left a comment

Choose a reason for hiding this comment

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

LGTM 🎉

The suggestion from randomguy also makes sense 👍

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

Reviewers

2 more reviewers

@trueberryless trueberryless trueberryless approved these changes

@randomguy-2650 randomguy-2650 randomguy-2650 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help!

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@urban-adeininger @astrobot-houston @trueberryless @randomguy-2650