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

markdownのFencedCodeに対応する #67

Merged
merged 12 commits into from
May 16, 2024
Merged

markdownのFencedCodeに対応する #67

merged 12 commits into from
May 16, 2024

Conversation

kfly8
Copy link
Contributor

@kfly8 kfly8 commented Apr 25, 2024 •
edited
Loading

Reverts #66

課題・背景

やったこと

  • Text::Markdown::Discountを利用して、Fenced Codeを置き換えました
  • ついでに、cpanfile.snapshotをGitHub Actionsで更新するようにも変更（ついでの割には変更内容が多くなってしまいましたが・・）

kfly8 and others added 12 commits April 25, 2024 20:41
@kfly8
Revert "Revert "markdownで書かれた翻訳は、GitHub Flavored Markdownを利用したい""
5ce8d0f
@kfly8
change Markdown::Perl to Markdent
64be108 
サーバーのperlが、5.18で、
Markdown::Perlが、5.26から対応していなかったため
@kfly8 @web-flow
Update cpanfile.snapshot and cpanfile.target
31d89be
@kfly8
markdownをhtmlに変換するユーティリティ追加
bbfa809
@kfly8
markdownの処理をユーティリティを使った形に置き換え
0891abe
@kfly8
compose: テストファイルの変更に追従してくれ
0c904f8
@kfly8
翻訳データのセットアップ処理が重いので、切り出し
b27eff0
@kfly8
手元のテストとCI用のテストを分ける / 翻訳データの生成が別処理になったため
8e5c6da
@kfly8
手で編集するファイルはvolumesにいれておく
1f8b876
@kfly8
Text::Markdown::Discount に変更してみる
27e4045 
Markdentでこんな感じで怒られたため。

>    # PSGI error: Can't locate object method "as_text" via package "Markdent::Event::StartLink" at /usr/src/app/local/lib/perl5/Markdent/Parser/SpanParser.pm line 902.
    # Failed test 'status is 200'
    # at t/endpoints.t line 92.
    # +-----+----+-------+
    # | GOT | OP | CHECK |
    # +-----+----+-------+
    # | 500 | eq | 200   |
    # +-----+----+-------+
    # Failed test at t/endpoints.t line 93.
    # Unicode::GCString is not installed, table may not display all unicode characters properly
    # +---------+---------------------------------+
    # | GOT     | CHECK                           |
    # +---------+---------------------------------+
    # | <UNDEF> | Perlに関係するその他の翻訳の一覧 - perldoc.jp |
    # +---------+---------------------------------+
@kfly8 @web-flow
Update cpanfile.snapshot and cpanfile.target
558d5b6
@kfly8
FENCEDCODEを有効にする
8a76fd3
@kfly8 kfly8 changed the title Revert "Revert "markdownで書かれた翻訳は、GitHub Flavored Markdownを利用したい"" markdownのFencedCodeに対応する May 15, 2024
@kfly8 kfly8 merged commit 8f339b8 into master May 16, 2024
2 checks passed
@kfly8 kfly8 deleted the revert-66-revert-65-fence branch May 16, 2024 00:01
This was referenced May 16, 2024
Merged
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kfly8