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

Added Japanese names #1

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

lauriegriffiths
Copy link

No description provided.

@titanix
Copy link

titanix commented Jul 26, 2021

You forgot to delete things from the Chinese copy-pasta in the table, like row "Zh = J, "ah" vowel. Jahng"

@SteveMCarroll
Copy link
Owner

Is it possible that our edits collided? I like a lot of the material here, but I suspect I added a Japanese section while you were writing it. Can you take a look at what I tossed up there and we can work together on what to incorporate? I had initially ignored vowel length as an example as I think you can get pretty close without worrying about it. I think your bit about syllable timing is quite good and concise. I was thinking of avoiding the topic, but now i'd like to incorporate since you described it so succinctly.

@SteveMCarroll
Copy link
Owner

also REALLY like the family name - given name point. I wonder if it's also worth tossing in something about the use of -san in any circumstance that isn't explicitly informal if the person is Japanese from Japan and not explicitly a close friend until invited to stop. I feel like I mostly do this in all business contexts and even though not strictly necessary in English, is a pretty good habit.

@lauriegriffiths
Copy link
Author

We must have been writing at the same time! I'm happy for you to take any parts from my version and add them in as you see fit!

It might be worth mentioning something about '-san', although it's another topic like name order where there is no real consistency or right answer.

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.

3 participants