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

Add Japanese support for wolfSSL-FIPS-Ready #129

Merged
merged 1 commit into from
Jun 12, 2024

Conversation

tamasan238
Copy link
Member

No description provided.

@kojo1 kojo1 self-requested a review May 26, 2024 21:53
theme:
name: null
custom_dir: ../mkdocs-material/material
language: en
Copy link
Contributor

Choose a reason for hiding this comment

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

language: jaでなくても大丈夫ですか?

Copy link
Member Author

Choose a reason for hiding this comment

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

ほかの製品のmkdocs-ja.ymlのlanguageに倣ってenとしていましたが,確かにjaのほうが望ましいですね.

a. このファイルだけjaに変更してcommit
b. 一旦enのままcommitし,他の製品とあわせてすべてlangをjaに修正するPRを作成
c. このファイルについてはjaに修正した上でcommitし,他の製品については別途PRを作成

いかがしましょうか.
jaに変更してビルドしたところ,以下の差分がありました.

  • pdf: 見た目上の差分なし(binary diffは差分ありと出力)
  • html: フォントサイズが異なる

Copy link
Contributor

@kojo1 kojo1 Jun 1, 2024

Choose a reason for hiding this comment

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

では、全部まとめてlanguageだけ変更する別PRとしましょうか。

Copy link
Member Author

Choose a reason for hiding this comment

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

承知しました.


ここでは、LinuxまたはmacOS環境でwolfSSL FIPS Ready GPL v3版を利用し、システムに共有ライブラリとしてインストールする手順を示します。

## ソースコードを解凍します
Copy link
Contributor

Choose a reason for hiding this comment

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

セッション名などの場合、"します" はないほうがすっきりするかと。(他も合わせてください)

@kojo1 kojo1 merged commit b7108e3 into wolfSSL:master Jun 12, 2024
2 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