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

markedjsのアップデート #1694

Open
3 tasks
Tracked by #1719
azu opened this issue Sep 3, 2023 · 3 comments
Open
3 tasks
Tracked by #1719

markedjsのアップデート #1694

azu opened this issue Sep 3, 2023 · 3 comments
Labels
Status: PR Welcome Pull Requestを歓迎する状態 Status: Proposal 提案段階の状態 Type: ユースケース
Milestone

Comments

@azu
Copy link
Collaborator

azu commented Sep 3, 2023

ユースケースの章のmarkedとそろそろだいぶ変わってきているのでアップデートする。
Deprecatedなオプションが消えたのでちょうどいいはず

TODO

@azu azu added Type: ユースケース Status: Proposal 提案段階の状態 Status: PR Welcome Pull Requestを歓迎する状態 labels Sep 3, 2023
@azu
Copy link
Collaborator Author

azu commented Dec 10, 2023

marked がバージョン上げすぎてちょっと扱うにくい気がする。

@azu azu mentioned this issue Dec 29, 2023
14 tasks
@azu azu added this to the v6(ES2024) milestone Feb 10, 2024
@azu azu changed the title markedjs v8へのアップデート markedjsのアップデート Feb 10, 2024
@azu
Copy link
Collaborator Author

azu commented Mar 23, 2024

@SotaYamaguchi さんお願いした。

ちょっとアップデートでどういう変更が必要かが見えてないので、アップデートしてみて文章やテストを変更しないといけない箇所を調べるところからやるのが良いと思います。

📝 ほかの変更と重なってしまう部分があるので、最初か最後にマージしないといけない

@SotaYamaguchi
Copy link

SotaYamaguchi commented Apr 20, 2024

  • markedのバージョンは最新の 12.X.X を使う
  • サンプルコードで利用する範囲で、markedのIFが大きく変わっているところはない
  • デフォルトの動作からidが消えているのでユニットテストを記述する · JavaScript Primer #jsprimerも変更が必要

  • marked側でサニタイズは責務外だから自分でしてねと警告されている。今回は外部ライブラリを用いたCLIアプリケーション作成の理解が目的。HtmlParserを作る際のセキュリティ面の注意喚起は本書では扱わなくてよさそう。
    • JavaScriptのライブラリやフレームワークの使い方を学ぶのが目的ではない

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: PR Welcome Pull Requestを歓迎する状態 Status: Proposal 提案段階の状態 Type: ユースケース
Projects
None yet
Development

No branches or pull requests

2 participants