Skip to content

Conversation

@bilhox
Copy link
Contributor

@bilhox bilhox commented Oct 5, 2024

This PR is an up-to-date review of my french translation created a year ago, I mainly changed some wording I was using which wasn't natural, and more importantly, I added a note at the end of the file saying when it was last reviewed.

@bilhox bilhox added the docs label Oct 5, 2024
@bilhox bilhox requested a review from a team as a code owner October 5, 2024 10:08
@bilhox bilhox requested a review from pmp-p October 5, 2024 10:12
@damusss
Copy link
Member

damusss commented Oct 5, 2024

@bilhox it's actually very similar to what I did, I only added an header line before. would you not agree it would be fitting?

@bilhox
Copy link
Contributor Author

bilhox commented Oct 5, 2024

@bilhox it's actually very similar to what I did, I only added an header line before. would you not agree it would be fitting?

yes, just add also an english version of the date like I did.

Copy link
Member

@MyreMylar MyreMylar left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link

@TheRealGabHas TheRealGabHas left a comment

Choose a reason for hiding this comment

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

I think all the changes are good, they make the reading process more natural.

Copy link
Member

@ankith26 ankith26 left a comment

Choose a reason for hiding this comment

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

Thanks!

@ankith26 ankith26 added this to the 2.5.2 milestone Oct 5, 2024
@ankith26 ankith26 merged commit 76adc02 into pygame-community:main Oct 5, 2024
@bilhox bilhox deleted the french-last-reviewed branch October 13, 2024 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants