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

[ja_JP] Fix "What's new in Hyperledger Fabric v2.x" #940

Merged
merged 1 commit into from
Jul 31, 2024

Conversation

takayuki-nagai
Copy link
Contributor

"whatsnew.md" is still as of v2.2, so this patch reflects modifications up to v2.5.
This patch is related to the issue #898.

@takayuki-nagai takayuki-nagai requested review from a team as code owners July 19, 2024 02:37
@shimos shimos self-assigned this Jul 30, 2024
Copy link
Contributor

@shimos shimos left a comment

Choose a reason for hiding this comment

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

@takayuki-nagai 翻訳ありがとうございました。レビューが遅くなり申し訳ありません。
一部、反映されていない部分があるようですので、確認をお願いいたします。

(以下の行数は、現在のPRの修正後のファイルのでの行数となります)

  • ll. 146-149 (古いバージョンに対する記述なので、実質的にはあまり意味がないですが) 表現が変わっています。
  • l.154 2.x系のバージョンが増えたためか、以降のSubsectionがSubsubsectionに変更されているところがありますので、反映をお願いいたします。
  • l.203 このSubsubsectionはタイトルのみ削除されています

Signed-off-by: takayuki-nagai <takayuki.nagai.nu@hitachi.com>

Modify document according to reviewer's comments.

Signed-off-by: takayuki-nagai <takayuki.nagai.nu@hitachi.com>
@takayuki-nagai
Copy link
Contributor Author

@takayuki-nagai 翻訳ありがとうございました。レビューが遅くなり申し訳ありません。 一部、反映されていない部分があるようですので、確認をお願いいたします。

(以下の行数は、現在のPRの修正後のファイルのでの行数となります)

* ll. 146-149 (古いバージョンに対する記述なので、実質的にはあまり意味がないですが) 表現が変わっています。

* l.154 2.x系のバージョンが増えたためか、以降のSubsectionがSubsubsectionに変更されているところがありますので、反映をお願いいたします。

* l.203 このSubsubsectionはタイトルのみ削除されています

コメントありがとうございます。上記ポイントを修正しpushし直しました。ご確認をお願いします。

Copy link
Contributor

@shimos shimos left a comment

Choose a reason for hiding this comment

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

@takayuki-nagai 修正ありがとうございました!LGTM

@shimos shimos merged commit db8f1fc into hyperledger:release-2.5 Jul 31, 2024
4 checks passed
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