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

ja: Translate Chapter 25 (Traits) #1436

Merged
merged 8 commits into from
Jan 4, 2024

Conversation

kantasv
Copy link
Collaborator

@kantasv kantasv commented Oct 28, 2023

Konnichiwa, JP translation folks #652. Here's a MR for Traits chapter. I'm open to suggestions and feedback :) Could you take a look at it? Thank you!

@kantasv
Copy link
Collaborator Author

kantasv commented Oct 28, 2023

FYI (You can stop reading if you don't use a MacBook)

Looks like the previous commit (a5f938f) had italic double quotes, which failed its Japanese Github Actions job.

When I copy and pasted snippets from original one to translation box in Poedit, my Mac automatically converted double quotes to italic ones. This is fancy but not the best for programmers, haha :) I have disabled the feature by following the guide: https://blog.piyo.tech/posts/2014-01-06-205101/

The double quotes were fixed by the commit, cb4b5d6.

@keiichiw
Copy link
Collaborator

keiichiw commented Nov 1, 2023

@HidenoriKobayashi would you mind reviewing this PR?

Copy link
Collaborator

@HidenoriKobayashi HidenoriKobayashi left a comment

Choose a reason for hiding this comment

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

@keiichiw Please review my comments.

@kantasv
Copy link
Collaborator Author

kantasv commented Nov 4, 2023

Thanks @HidenoriKobayashi for great feedback, espesh checking readability of my translation. I've applied most of the suggestions except two --- I left two comments. Could you have a look at them? Have a good holiday!

Shortcuts to open threads to save your minuets

Copy link
Collaborator

@keiichiw keiichiw left a comment

Choose a reason for hiding this comment

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

Thanks for your PR! Here are my review comments

@kantasv
Copy link
Collaborator Author

kantasv commented Nov 12, 2023

Thanks @keiichiw -san, I have applied suggestions; I added several footnotes in the paragraphs with parenthesis, e.g., "trait bounds" translation discussions. Could you review the diff? Any feedback is appreaciated. Cheers!

@kantasv
Copy link
Collaborator Author

kantasv commented Nov 12, 2023

Ah, some of the github actions checks have failed. I'll take a look into this.

@mgeisler
Copy link
Collaborator

Ah, some of the github actions checks have failed. I'll take a look into this.

Sorry, that is my mistake... it'll be fixed with #1472!

@mgeisler
Copy link
Collaborator

Ah, some of the github actions checks have failed. I'll take a look into this.

Sorry, that is my mistake... it'll be fixed with #1472!

You are welcome to review that PR so it can be auto-merged. That will unblock the build for you and others 🙂 Then we can clean up any left-over problems afterwards.

@kantasv
Copy link
Collaborator Author

kantasv commented Nov 12, 2023

Ah, some of the github actions checks have failed. I'll take a look into this.

Sorry, that is my mistake... it'll be fixed with #1472!

You are welcome to review that PR so it can be auto-merged. That will unblock the build for you and others 🙂 Then we can clean up any left-over problems afterwards.

OK, I try to take a look into this!

@keiichiw keiichiw self-requested a review November 14, 2023 12:46
mgeisler pushed a commit that referenced this pull request Dec 10, 2023
Hi, this is a tiny MR to add glossary entry for "trait bound". Could you
someone review it? I'm open to suggeston, too. Thank you!
Context:
#1436 (comment)
rickyclarkson pushed a commit to rickyclarkson/comprehensive-rust that referenced this pull request Dec 11, 2023
Hi, this is a tiny MR to add glossary entry for "trait bound". Could you
someone review it? I'm open to suggeston, too. Thank you!
Context:
google#1436 (comment)
@mgeisler
Copy link
Collaborator

Hey all, thanks for keeping working on the translation! I think most comments above were incorporated, so perhaps we can merge this?

Copy link
Collaborator

@keiichiw keiichiw left a comment

Choose a reason for hiding this comment

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

Yeah, this looks good to me now! Approved.

@kantasv
Copy link
Collaborator Author

kantasv commented Jan 4, 2024

Thanks @mgeisler for the follow up, and @keiichiw for approval. Merging.

@kantasv kantasv merged commit b1f6851 into google:main Jan 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants