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.

Sign up for GitHub

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

Jump to bottom

markedjsのアップデート #1694

Closed
3 tasks
Tracked by #1719
azu opened this issue Sep 3, 2023 · 5 comments · Fixed by #1760
Closed
3 tasks
Tracked by #1719

markedjsのアップデート #1694

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

Comments

@azu
Copy link
Collaborator

azu commented Sep 3, 2023 •
edited
Loading

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

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

azu commented Dec 10, 2023

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

@azu azu mentioned this issue Dec 29, 2023
Closed
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 azu mentioned this issue Feb 10, 2024
Closed
4 tasks
@azu
Copy link
Collaborator Author

azu commented Mar 23, 2024

@SotaYamaguchi さんお願いした。

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

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

@SotaYamaguchi
Copy link

SotaYamaguchi commented Apr 20, 2024 •
edited
Loading

  • markedのバージョンは最新の 12.X.X を使う
  • サンプルコードで利用する範囲で、markedのIFが大きく変わっているところはない

  • デフォルトの動作からidが消えているのでユニットテストを記述する · JavaScript Primer #jsprimerも変更が必要

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

    • JavaScriptのライブラリやフレームワークの使い方を学ぶのが目的ではない

@azu azu mentioned this issue Aug 3, 2024
Merged
@azu
Copy link
Collaborator Author

azu commented Aug 3, 2024

他の Node.jsの変更はマージしました #1719

@azu
Copy link
Collaborator Author

azu commented Aug 25, 2024

これそろそろ進める

@azu azu mentioned this issue Aug 25, 2024
Merged
@azu azu linked a pull request Aug 25, 2024 that will close this issue
fix(nodecli): update to marked@14 #1760
Merged
@azu azu closed this as completed in #1760 Aug 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Status: PR Welcome Pull Requestを歓迎する状態 Status: Proposal 提案段階の状態 Type: ユースケース
Projects
None yet
Milestone
v6(ES2024)
Development

Successfully merging a pull request may close this issue.

fix(nodecli): update to marked@14 asciidwango/js-primer
2 participants
@azu @SotaYamaguchi