Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Translate string.rst:777 to 786#717

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
mattwang44 merged 2 commits intopython:event-1125fromproudofsimin:translate
Jan 4, 2024

Conversation

proudofsimin
Copy link
Contributor

Hi, All
Please review. Thank you.

Copy link
Collaborator

@rockleonarockleona left a comment

Choose a reason for hiding this comment

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

Review Part 1

Copy link
Collaborator

@ken71301ken71301 left a comment

Choose a reason for hiding this comment

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

用詞與空格需修正

@ken71301ken71301 changed the base branch from3.12 toevent-1125November 25, 2023 08:45
Copy link
Collaborator

@ken71301ken71301 left a comment

Choose a reason for hiding this comment

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

仍有部分翻譯錯誤

@mattwang44
Copy link
Collaborator

rebase

@rockleona
Copy link
Collaborator

@proudofsimin rebase 需要協助的話都可以跟我說喔
也可以到 Discord 私訊我

@proudofsimin
Copy link
ContributorAuthor

@proudofsimin rebase 需要協助的話都可以跟我說喔
也可以到 Discord 私訊我

Sry for the late reply. I’m pretty occupied these days. I’m not sure what that means but will try to work on it next week. Many thanks!

@proudofsimin
Copy link
ContributorAuthor

proudofsimin commentedDec 17, 2023
edited
Loading

Hi,
I googled what rebase means, but I'm not sure if there's anything else I should've done other than regular git check-in. Please let me know if I have use the command correctly. Thank you.

@mattwang44
Copy link
Collaborator

mattwang44 commentedDec 18, 2023
edited
Loading

Hi@proudofsimin
If you're referring to therebase comment I left last time, it means I've done agit rebase (doc andvisualization) for the branch of this PR. Wheneveryour branch has any conflict with the target branch (like now), you'll need to resolve withgit rebase ... orgit merge ... command. We prefer the former as the commit history would be more readable, which eases the pain for reviewers. You canresolve the conflict on the GitHub UI, but it runs agit merge ... command under the hood.

Feel free to pin us on the Discord server if you have any questions!

@mattwang44
Copy link
Collaborator

mattwang44 commentedJan 3, 2024
edited
Loading

@proudofsimin
最近要 wrap up 我們 11 月活動的所有 PR 了,想請問你有空來處理這個 PR 嗎?
還是我能否幫你來修正剩下未完成的部分呢?

Copy link
Collaborator

@ken71301ken71301 left a comment

Choose a reason for hiding this comment

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

fix the sentence in L:1053 and we are good to go.

mattwang44 reacted with thumbs up emoji
@proudofsimin
Copy link
ContributorAuthor

@proudofsimin 最近要 wrap up 我們 11 月活動的所有 PR 了,想請問你有空來處理這個 PR 嗎? 還是我能否幫你來修正剩下未完成的部分呢?

好,我好感動謝謝!!我自己在local端試了好幾次修不好我的environment弄不上去...非常感謝

@mattwang44
Copy link
Collaborator

@proudofsimin 好呀,我就先處理這個 PR,如果是對於 git or github 操作上遇到問題的話都歡迎在 discord 上找我們討論歐!

@mattwang44mattwang44 merged commit0533ee6 intopython:event-1125Jan 4, 2024
@proudofsimin
Copy link
ContributorAuthor

@proudofsimin 好呀,我就先處理這個 PR,如果是對於 git or github 操作上遇到問題的話都歡迎在 discord 上找我們討論歐!

好的超級感謝,我其實之前沒用過discord所以一直都不敢發言...造成您們的困擾了非常不好意思QQ

@mattwang44
Copy link
Collaborator

@proudofsimin 沒關係的 沒有造成困擾~
如果不敢公開發言的話也可以考慮私訊我 (my id:mattwang44),不用怕麻煩到我(只是我比較忙的話就會比較晚回而已XD)

mattwang44 added a commit that referenced this pull requestJan 26, 2024
* Translate rst777:786.* fix(library/string): refine translation---------Co-authored-by: Matt Wang <mattwang44@gmail.com>
mattwang44 added a commit that referenced this pull requestJan 27, 2024
* Translate rst777:786.* fix(library/string): refine translation---------Co-authored-by: Matt Wang <mattwang44@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@mattwang44mattwang44mattwang44 left review comments

@ken71301ken71301ken71301 left review comments

@rockleonarockleonaAwaiting requested review from rockleona

Assignees

@proudofsiminproudofsimin

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@proudofsimin@mattwang44@rockleona@ken71301

[8]ページ先頭

©2009-2025 Movatter.jp