Improve README #47
Draft
Improve README #47
+57
−5
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
そういえばnodeとnpmのどのバージョンで動くか書いときたいですね |
makenowjust
reviewed
Dec 18, 2020
Comment on lines
+8
to
+10
| セキュリティ・キャンプについては[こちら](https://www.ipa.go.jp/jinzai/camp/) | ||
| セキュリティ・キャンプ全国大会2020 オンラインについては[こちら](https://www.ipa.go.jp/jinzai/camp/2020/zenkoku2020_index.html) | ||
| 講義「ReDoSの検出プログラムの作成とOSSへの適用」については[こちら](https://www.ipa.go.jp/jinzai/camp/2020/zenkoku2020_program_list.html#list_s-z3) |
There was a problem hiding this comment.
末尾に空白を入れて改行するようにしてるみたいですが、このようにしたいのであれば素直にリストを使った方が良いと思います。
makenowjust
reviewed
Dec 18, 2020
Comment on lines
+61
to
+63
| ## 動作に必要なもの | ||
|
|
||
| - npm |
There was a problem hiding this comment.
これが末尾あるのは違和感があります。あまり重要でもないので削除するか、そうでなければ「使い方」の先頭に持っていった方がいいと思います
|
動作に必要なもの (requirements) は個人的には要らない気がするのですがどうでしょう？ node や npm のバージョンに関する記述はしないことが多いように思う（これは他の人が言っていたことの受け売りなんですが、Node.js は後方互換性に優れているためユーザが latest を使わない意義が無いため） ……のですが、今試してみたら Node.js 10 系では 10 系はサポート期間が 2021-04-01 までらしいので一応 12 系以上が対象であることは書いておくべきか……。 |
|
自分が最初に、「動作に必要なもの npm」を特に深い意味はなく書いてしまったので消してもらっても大丈夫です。 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
解決する Issue
改善内容
各自参加記を書いたりするにあたってこのリポジトリを引用することもあるかと思うので、README をしっかりしたものに書き換えます。（もう遅かったらごめんなさい……）
基本的に #35 の体裁を整えたのと、序文は #44 をもとにしています。
発表資料の部分には発表スライドへのリンク付きサムネイルを貼ります。