Skip to content

Release work ( ~ October 1, 2018)#295

Merged
137 commits merged intomasterfrom
develop
Sep 14, 2018
Merged

Release work ( ~ October 1, 2018)#295
137 commits merged intomasterfrom
develop

Conversation

@daisukenishino
Copy link
Copy Markdown
Contributor

@daisukenishino daisukenishino commented Jul 31, 2018

Release of 02-30 version (within 1 October 2018)

memo of test cases.

Obtain diff in order to confirm the added / changed function, and add test cases as following.

  • Roughly be passed through communication control (WebAPI) .
  • Validate jws from asp.net core.
  • , etc.

Comment thread root/Readme.ja.md Outdated
```txt
C:\root\programs\C#\Frameworks\Infrastructure\Public\Dll
```
「root」フォルダを、C ドライブ直下にコピーしてください。そうでない場合は、Windows のファイルパスの最大文字長を超え、ビルドに失敗することがあります。
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@daisukenishino

  • 「そうでない場合は」→「C ドライブ直下以外にコピーすると」
  • 「Windows のファイルパスの最大文字長を超え」→「Windows のファイルパスの最大文字長の制限により」

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@yusukemaegawa
了解。上記のように修正します。

Comment thread root/README.md Outdated

### Obtaining data provider, Exporting prodedures, Attaching to license
Obtain and export the data provider, which Open Touryo supports, on your own.
### [How to create NuGet packages](https://github.com/OpenTouryoProject/OpenTouryo/wiki/HowToCreateOpenTouryoNuGetPackages)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@daisukenishino

見出しにはリンクを張らないでください。

### How to create Open Touryo NuGet packages

For the method of creating Open Touryo NuGet packages, see [this article](https://github.com/OpenTouryoProject/OpenTouryo/wiki/HowToCreateOpenTouryoNuGetPackages).

日本語版も同様に修正してください。

Comment thread root/README.md Outdated
You can confirm the structure of Open Touryo and the specification of each feature.
- [Tutorial](https://github.com/OpenTouryoProject/OpenTouryoDocuments/tree/master/documents/2_Tutorial)
You can see the *first step guide* of Open Touryo.
## Other end matter
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@daisukenishino

"end matter" は「後付け」という意味ですが、日本語版だと「その他」となっており、意味が違うと思います。

「その他特記事項」という意味なら、"Other items of note" だと思います。

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@yusukemaegawa

ココを参考にしましたが、
http://www.nakanihon.co.jp/gijyutsu/Shimada/bunsyo/index.htm

どちらにしましょうか。

  • その他の項目, Other end matter
  • その他特記事項, Other items of note

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@daisukenishino

いろいろ調べてみても、"end matter" は「後付け」と出てきます。「後付け」は、「本文の後に続く文章」という意味なので、内容的に「特記事項」のほうが正しいと思います。

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@yusukemaegawa
すいません、ココ漏れてましたので修正しました。

Comment thread root/README.md Outdated
Obtain and export the data provider, which Open Touryo supports, on your own.
### [How to create NuGet packages](https://github.com/OpenTouryoProject/OpenTouryo/wiki/HowToCreateOpenTouryoNuGetPackages)

### Obtaining libraries, Exporting control prodedures, Attaching to license
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@daisukenishino

文章の途中で大文字を使うのは、文法上正しくありません。

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@yusukemaegawa
そこは、元々のトコロですが、修正しました。
NuGetはイイと思いますが、Package Managerもですか?

Comment thread root/README.md Outdated

### Obtaining libraries, Exporting control prodedures, Attaching to license
- Since libraries that can be obtaining from Package Manager do not need to be bundled, they can be excluded from export control.
- For libraries that are bundled and exported, it is necessary to the exporting control procedure and attaching to license.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@daisukenishino

The libraries that can be obtained from package manager, that is NuGet or npm, are not bundled in Open Touryo. So, you don't have to export the libraries.
You have to obtain and export the other libraries, that is the libraries that can not be obtained from package manager, on your own. And you have to attach the libraries to license.

ライセンスに何を添付しなければならないのかがわからないので、最後の一文はあまり自信がありません。

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@yusukemaegawa
最後の一文ですが、同梱時なので、「(Open棟梁の)ライセンスに(使用するライブラリの)ライセンスを添付する。」だと思います。

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@daisukenishino

であれば、先ほどの文章を、「(Open棟梁の)ライセンスに(使用するライブラリの)ライセンスを添付する。」の意味になるように修正して、文章全体を修正ください。

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@yusukemaegawa
修正したので確認お願いします。

Comment thread root/README.md Outdated
- Since libraries that can be obtaining from Package Manager do not need to be bundled, they can be excluded from export control.
- For libraries that are bundled and exported, it is necessary to the exporting control procedure and attaching to license.
- The libraries that can be obtained from package manager, that is NuGet or npm, are not bundled in Open Touryo. So, you don't have to export the libraries.
- If necessary, you have to obtain and export the other libraries, that is the libraries that can not be obtained from package manager, on your own. In this case, you have to attach the license of libraries to be used to the license of Open Touryo.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@daisukenishino

attach the license of libraries → attach the license of the libraries

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@yusukemaegawa
修正したので確認お願いします。

@ghost ghost merged commit 4412d42 into master Sep 14, 2018
This pull request was closed.
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