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

翻訳についての提案 #379

Merged
merged 5 commits into from
Nov 14, 2022
Merged

翻訳についての提案 #379

merged 5 commits into from
Nov 14, 2022

Conversation

sumomoneko
Copy link
Contributor

これは読み進めた中で書きためた、個人的な好き嫌いの範囲に入るような変更の提案です。

修正・却下など、見ていただけるとうれしいです。

Closes #378

Copy link
Owner

@stzn stzn left a comment

Choose a reason for hiding this comment

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

ありがとうございます!一点だけコメントしました。

language-guide/properties.md Outdated Show resolved Hide resolved
Co-authored-by: shiz <35151927+stzn@users.noreply.github.com>
@sumomoneko
Copy link
Contributor Author

レビューありがとうございます!

了解です。指摘された内容をコミットしました。

Copy link
Owner

@stzn stzn left a comment

Choose a reason for hiding this comment

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

LGTMです!ありがとうございますmm明日まで待って他にご意見なければマージします。

@stzn stzn merged commit 87d42d6 into stzn:master Nov 14, 2022
@stzn
Copy link
Owner

stzn commented Nov 14, 2022

ごめんなさい。バタバタしていてマージするのを失念していましたmm
ありがとうございました!

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.

翻訳についての提案
2 participants