search

kzrnm / ac-library-csharp

41 stars 5 forks source link

add SourceCodeEmbedder #65

Closed kzrnm closed 4 years ago

kzrnm commented 4 years ago

50 では Expander 全体を Pull Request の対象としていましたが、 SourceCodeEmbeder だけとさせていただきたいです。

Expander の管理を考えるとコードオーナーは私である方が良く、nuget の依存関係によって導入するような仕組みとなれば良いかと思いました。

AtCoder.Embbed namespace は、3rd-party のライブラリでの使用もできるので ac-library-cs に含めてしまう方が良いかなと思います。

kzrnm commented 4 years ago

https://github.com/naminodarie/ac-library-cs-expander

Expander はこちらで配布することにします。

fairy-lettuce commented 4 years ago

本質的ではありませんが、Embed + -erの活用形はEmbederではなくEmbedderではありませんか……？

kzrnm commented 4 years ago

Embedder に修正しました。

また、embedder はライブラリ本体ではないので tools ディレクトリに移動しました。

fairy-lettuce commented 4 years ago

修正ありがとうございます！

kzrnm commented 4 years ago

Embedder をNuget化できそうなので閉じます。