nuitsjp / CopyToMarkdownAddIn

Add-In for copying from Excel to Markdown
MIT License
172 stars 15 forks source link

表内の改行への対応 #3

Closed chie8842 closed 8 years ago

chie8842 commented 8 years ago

こんにちは。便利に使用させていただいているのですが、表内での改行の際、markdownでは、\r\nを
に置換しておかないと、表が壊れてしまいます。 できればご対応いただきたいです。 いったんforkしてソースコード変更してみたのですが、恥ずかしながらビルドの手順がわからず、動作確認できていません。 もしビルド方法をお教えいただければ、プルリク投げさせていただきたいです。

nuitsjp commented 8 years ago

お~なるほど。 ビルドは証明書が必要で、ちょっとこれはお渡しできませんのでプロジェクトのプロパティから仮証明書を作成してテストしていただけますでしょうか? 下図のあたりです。

0137

プルリクお待ちしています!

chie8842 commented 8 years ago

すみません、Win10にvisualstudio2015をインストールしてみましたが、 The following projects are not supported by eigher this edition of Visual Studio or the version of Windowsとでてしまいます。。。 お手数ですが、ビルド環境をお教えいただけないでしょうか。

nuitsjp commented 8 years ago

OSはWindows 10 64bit のアニバーサリーエディション。(OSは問題ない気がします) VSは2015のUpdate 3です。
拡張機能としてOffice Developer Tools for Visual Studio 2015が入っている必要があります。 これはVSインストーラのカスタム 他は特にないと思いますが。。。

0138

Web Platform Installerから入れられると思います。 0139

chie8842 commented 8 years ago

すみません、できました。プルリク送りました。 スクリーンショットなど丁寧にありがとうございました。

nuitsjp commented 8 years ago

RPありがとうございます! RP側にも書きましたが、ヘッダー側にも同様のロジックを適用して、且つ処理をメソッドに切り出したいです。 こちらで対応しても構いませんが、よかったらRP送ってもらえると嬉しいのですが、どういたしましょう?

chie8842 commented 8 years ago

あ!たしかに。すみません、headerの方は修正しました。 わたしC#はあまり書きなれていないこともあり、切り出し単位の好みなどもあるかと思いますので、 一旦マージいただいた後に変更していただいたほうがよいかな、と考えています。

nuitsjp commented 8 years ago

OKです。いったんマージしました。 後はこちらで対応します。 リリースまで少々お待ちください。 ご協力ありがとうございました!