Skip to content

Conversation

@MekalBoy
Copy link

Chapter 9 was translated. With this being an initial draft, I am looking for feedback on whether or not the wording is appropriate.

Chapter 9 was translated.
Co-authored-by: Alexandru Radovici <[email protected]>
@MekalBoy MekalBoy marked this pull request as draft January 20, 2023 21:41
The one-liner solutions present in the code review suggestions have been added.

Co-authored-by: Alexandru Radovici <[email protected]>
@MekalBoy
Copy link
Author

I hope to make more time to properly fix or otherwise touch up the rest of the file after Monday, at most by next week's time. Until then I have to dedicate my time towards studies.

@MekalBoy MekalBoy marked this pull request as ready for review January 23, 2023 14:58
The PI joke was replaced by the spanish version's joke.
Changed translation to better match other chapters.
Copy link

@alexandruradovici alexandruradovici left a comment

Choose a reason for hiding this comment

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

Send this pull request to upstream and place a link to it.

@MekalBoy
Copy link
Author

Added a PR Upstream:
richardanaya#458

@alexandruradovici alexandruradovici added the upstream This pull request was opened in the upstream repository label Jan 25, 2023
@alexandruradovici
Copy link

What is your full name and group?

@MekalBoy

This comment was marked as off-topic.

@alexandruradovici alexandruradovici changed the title Chapter 9 Romanian Translation Chapter 9 Romanian Translation 🇷🇴 Jan 27, 2023
@alexandruradovici
Copy link

I suggest adding the Romanian flag 🇷🇴 (copy paste) to the upstream pull request title.

richardanaya pushed a commit to richardanaya/tour_of_rust that referenced this pull request Feb 4, 2023
* Chapter 9 Romanian Translation

Chapter 9 was translated.

* Update lessons/ro/chapter_9.yaml

Co-authored-by: Alexandru Radovici <[email protected]>

* Apply part of suggestions from code review

The one-liner solutions present in the code review suggestions have been added.

Co-authored-by: Alexandru Radovici <[email protected]>

* Additional translation fixes

This commit resolves the following suggestions:
UPB-CS-OpenSourceUpstream#5 (comment)
UPB-CS-OpenSourceUpstream#5 (comment)
UPB-CS-OpenSourceUpstream#5 (comment)
UPB-CS-OpenSourceUpstream#5 (comment)
UPB-CS-OpenSourceUpstream#5 (comment)

* Replaced PI joke

The PI joke was replaced by the spanish version's joke.

* Update translation

Changed translation to better match other chapters.

---------

Co-authored-by: Alexandru Radovici <[email protected]>
@alexandruradovici
Copy link

@MekalBoy Please select a timeslot for the interview https://doodle.com/meeting/participate/id/eZ8gOOEb.

README has been updated to include Romanian chapters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

upstream This pull request was opened in the upstream repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants