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

Sync with upstream #128

Merged
merged 10 commits into from
Nov 25, 2024
Merged

Sync with upstream #128

merged 10 commits into from
Nov 25, 2024

Conversation

nips-ja-sync[bot]
Copy link

@nips-ja-sync nips-ja-sync bot commented Nov 24, 2024

No description provided.

@s3-odara s3-odara merged commit b1ddf7a into main Nov 25, 2024
1 check passed
@s3-odara s3-odara deleted the sync branch November 25, 2024 09:18
@@ -6,7 +6,7 @@ NIP-24

`draft` `optional`

このNIPはイベントに加えられる追加の任意のフィールドを定義する
このNIPはどこにも定義されていないが_事実上_標準になっているイベントに加えられる追加の任意のフィールドや、それ自体のNIPを作るに値せず、他のNIPにも受け入れられない些細な仕様を追跡する
Copy link
Member

Choose a reason for hiding this comment

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

LGTMですが、nitsとして、「は、」として主語の範囲を明示すると読みやすくなるかなと思いました。

Suggested change
このNIPはどこにも定義されていないが_事実上_標準になっているイベントに加えられる追加の任意のフィールドや、それ自体のNIPを作るに値せず、他のNIPにも受け入れられない些細な仕様を追跡する。
このNIPは、どこにも定義されていないが_事実上_標準になっているイベントに加えられる追加の任意のフィールドや、それ自体のNIPを作るに値せず、他のNIPにも受け入れられない些細な仕様を追跡する。

Copy link
Contributor

Choose a reason for hiding this comment

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

あー確かにそのほうがいいですね。

Copy link
Contributor

Choose a reason for hiding this comment

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

ありがとうございます。
PR開きました
#129

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.

8 participants