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

#issue 235 ->Add Chinese translation #240

Merged
merged 3 commits into from
Aug 29, 2024
Merged

Conversation

Skydrgon
Copy link
Contributor

@Skydrgon Skydrgon commented Aug 17, 2024

#235
Add Chinese translation for README.md, the file is README-CN.md.
Updating help related to the nick and kick commands.

Not sure how to include a link to README-CN.md in README.md, do I need to commit this branch first and then reference it via markdown later?

Updating help related to the nick and kick commands.

Signed-off-by: Skydrgon <[email protected]>
README.md Outdated Show resolved Hide resolved
README.md Outdated
@@ -128,7 +134,7 @@ The rest
pwd Show your current room
shrug ¯\_(ツ)_/¯
```

tips: `kick` can help kick out your previous id if the nickname is taken due to network delays.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Great idea to mention that. Maybe adding an example in a code block could be useful.

Copy link
Owner

Choose a reason for hiding this comment

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

yeah good to include but imo a code block is not needed

README.md Outdated Show resolved Hide resolved
README-CN.md Outdated
在聊天室中:
OLDnick> nick NEWnick
NEWnick:
```
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Took both of your comments and updated them.
Removed code block about nick.

README.md Outdated Show resolved Hide resolved
@quackduck
Copy link
Owner

Looks good, thank you so much @Skydrgon!

@quackduck quackduck merged commit 226f630 into quackduck:main Aug 29, 2024
2 checks passed
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.

3 participants