naoya / md2inao

Convert markdown to inao-format for WEB+DB PRESS
http://md2inao.bloghackers.net/
211 stars 33 forks source link

#120 のPHP Markdown Extra footnote記法に対応 #121

Closed takahashim closed 8 years ago

takahashim commented 8 years ago

120 に対応してみました。

Text::Markdown::HoedownはHOEDOWN_EXT_FOOTNOTESオプションでfootnote記法をサポートしているようですが、これは脚注番号と脚注本体を分離して出力するタイプであり、本文中への埋め込みは難しそうだったため、自力で対応しています。

対応する脚注がない場合はエラーにしておいた方がよいかと思うのですが、Perlは久しぶりすぎてよく分からないので警告のみにしてあります(どう書くのがよいんでしょうか…?)。

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-0.3%) to 96.061% when pulling d8cd096a7f1d48057bd16641cd2feb91988fd9c6 on takahashim:footnotes into e4f311a97c37ac6892507722eac54f23e7272cca on naoya:master.

inao commented 8 years ago

@takahashim

ありがとうございます!

以下のInDesignタグ付きテキストへの変換のテストも追加していただけますでしょうか? 私のローカルではテストも通りました。

https://github.com/naoya/md2inao/commit/00e0b1976287b26430737d7747517f15d8d70743

(すみません、こういった別リポジトリからのPull Requestの場合の、コミットやさらなるPull Requestの方法がわからなかったので、naoya/md2inaoにpushしています)

@naoya

お手数ですがもろもろご確認いただけますでしょうか?

takahashim commented 8 years ago

@inao これPRの方に取り込んでおきますね

inao commented 8 years ago

@takahashim

ありがとうございます!

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-0.3%) to 96.061% when pulling 49118b0ae5e1a22ba9bd53b2be2eae7ff1da110c on takahashim:footnotes into e4f311a97c37ac6892507722eac54f23e7272cca on naoya:master.

takahashim commented 8 years ago

@inao すみません、元のサンプルが対応できてなかった(1行に複数個注釈があった時先頭しか対応できてなかった)ので修正しました。

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-0.3%) to 96.061% when pulling 04c8f00048f329ad17a5872f57670444e7eb3ac1 on takahashim:footnotes into e4f311a97c37ac6892507722eac54f23e7272cca on naoya:master.

inao commented 8 years ago

@takahashim

元のサンプルが対応できてなかった(1行に複数個注釈があった時先頭しか対応できてなかった)ので修正しました。

おお、ありがとうございます!

naoya commented 8 years ago

ちょいと実装箇所がグローバルすぎるというか、この手のテキスト全部みて置換、みたいなのを安易にいれてくと、このライブラリの構造は一体何だったんだ、みたいなところがあります。

もう少しスコープの狭い箇所に実装があったほうがいいかなと思いましたが、それは後々リファクタリングする方向で。いったんテストも通ってるようですしマージします。

inao commented 8 years ago

@naoya

ご確認とマージありがとうございます!