Hexyll.Core.Identifier をリファクタリングする about hexirp-hakyll HOT 16 CLOSED

hexyll-core  >     [ 1] runRules:        FAIL
hexyll-core  >       unit-test/Hexyll/Core/RulesTest.hs:45:
hexyll-core  >       expected: fromList [example.md,russian.md,example.md (metadataMatch),example.md (mv1),russian.md (mv1),example.md (mv2),russian.md (mv2),example.md (nav),example.md (raw),russian.md (raw)]
hexyll-core  >        but got: fromList []

ここのテスト結果を見るに、 setVersion は思ったより使われているようだ。これは (raw) の部分が version によるアノテーションである。

from hexirp-hakyll.

取り敢えず Path へ移行する。どのようにして使われているのかはまだわかってないのでエラーになるかもしれない。

from hexirp-hakyll.

Binary インスタンスの書き方分からん。

from hexirp-hakyll.

mzero とか使うのはいいの？

from hexirp-hakyll.

ソーソコードを読んだ結果、きちんと失敗として扱われるらしい。このくらいの情報がどこにもないのちょっとあかん。「 Binary はこんな感じにインスタンス書くんですよ」が公式ドキュメントにしかなくて、さらに「 Alternative なので失敗も扱えますよ」という情報がどこにもない

from hexirp-hakyll.

Path が Binary のインスタンスになっていないのに少しイラっと来たけど、これは Path を含むレコードを Binary のインスタンスにしようとした時、今ある関数の組み合わせで何とかなるからなのかな。何とかならない NFData はきちんとインスタンスになっているし。

from hexirp-hakyll.

setVersion に Maybe (First String) という Monoid 構造をいれようとしたけど、これは First (Maybe String) と同じだった。

from hexirp-hakyll.

違った

from hexirp-hakyll.

"monoid builder" でググったら思いかけず kazu-yamamoto 氏の「 Blaze Builder のひみつ」というドキュメントを見つけてしまった。

from hexirp-hakyll.

https://kazu-yamamoto.hatenablog.jp/entry/2018/11/29/155311

"monoid configuration" で出てきたこれが正解

from hexirp-hakyll.

ここから Identifier は相対パスが格納されると仮定されていることが分かる。

from hexirp-hakyll.

getIdentVersion は Identifier.Pattern でしか使われない。

from hexirp-hakyll.

https://haskell-haddock.readthedocs.io/en/latest/markup.html

Lists of the same type don’t have to be separated by a newline...

from hexirp-hakyll.

getIdentVersion と setIdentVersion は internal にしたほうがいいのでは……？

from hexirp-hakyll.

https://hackage.haskell.org/package/QuickCheck-2.13.2/docs/Test-QuickCheck-Modifiers.html

まとめモジュールで再エクスポートされているので、このモジュールをインポートする必要はないよとドキュメントに書いてくれるのありがたい。

from hexirp-hakyll.

よし、テストの実装終わった

from hexirp-hakyll.