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

Fix German translation of the PHP 8.3 release page #925

Merged
merged 1 commit into from
Jan 19, 2024

Conversation

TimWolla
Copy link
Member

The current translation is frankly terrible. It contains several spelling and grammar mistakes and it is very liberal in the translation, completely rephrasing (and dropping) important aspects of the English original.

The updated translation matches the English original pretty closely. For the ext/random and Override stuff I was the original author of the English original.

/cc @msamesch @joshuaruesweg as the maintainers of doc-de. I've attempted to use the terminology that is used within the German documentation (e.g. Elternklasse instead of Basisklasse), I'd appreciate another pair of eyes :-)


see #902

Copy link

🚀 Commit c021a92 Deployed on https://web-php-pr-925.preview.thephp.foundation

@TimWolla
Copy link
Member Author

Note: https://www.php.net/releases/8.2/de.php / #903 is similarly terrible, but as it is not as prominent, given it's not the newest release, I did not yet bother fixing the translation. A short term fix might be simply deletion the German version for 8.2 / reverting #903.

@saundefined saundefined self-assigned this Jan 15, 2024
@saundefined
Copy link
Member

I'm going to merge it, if there'll any improvement, let's do it in the new PR :)

Thanks a lot!

@saundefined saundefined merged commit e3b39ca into php:master Jan 19, 2024
3 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.

2 participants