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.

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

Add Travius and appveryor test code and Fix epubv2 #1300

Merged
merged 3 commits into from
May 22, 2019

Conversation

mitsuo0114
Copy link
Contributor

@mitsuo0114 mitsuo0114 commented May 21, 2019

"review-init hello2 --epub-version 2"で作成した初期ディレクトリでepubmakerを走らせると、
以下のエラーで落ちたため、テストの追加(travis / appveryorでのreview-init + epubcheckによる形式チェック)と共に修正をPRさせていただきます。

エラービルド:
https://travis-ci.org/mitsuo0114/review/builds/535126919

$ ruby bin/review-init hello2 --epub-version 2 && cd hello2 && ruby ../bin/review-epubmaker config.yml && epubcheck book.epub
WARN: hello2.re:1: headline is empty.
Traceback (most recent call last):
	14: from ../bin/review-epubmaker:16:in `<main>'
	13: from /Users/travis/build/mitsuo0114/review/lib/review/epubmaker.rb:70:in `execute'
	12: from /Users/travis/build/mitsuo0114/review/lib/review/epubmaker.rb:105:in `execute'
	11: from /Users/travis/build/mitsuo0114/review/lib/review/epubmaker.rb:199:in `produce'
	10: from /Users/travis/build/mitsuo0114/review/lib/epubmaker/producer.rb:204:in `produce'
	 9: from /Users/travis/build/mitsuo0114/review/lib/epubmaker/epubv2.rb:127:in `produce'
	 8: from /Users/travis/build/mitsuo0114/review/lib/epubmaker/epubcommon.rb:402:in `produce_write_common'
	 7: from /Users/travis/build/mitsuo0114/review/lib/epubmaker/epubcommon.rb:402:in `open'
	 6: from /Users/travis/build/mitsuo0114/review/lib/epubmaker/epubcommon.rb:402:in `block in produce_write_common'
	 5: from /Users/travis/build/mitsuo0114/review/lib/epubmaker/producer.rb:107:in `opf'
	 4: from /Users/travis/build/mitsuo0114/review/lib/epubmaker/epubv2.rb:25:in `opf'
	 3: from /Users/travis/build/mitsuo0114/review/lib/epubmaker/epubv2.rb:37:in `opf_metainfo'
	 2: from /Users/travis/build/mitsuo0114/review/lib/epubmaker/epubv2.rb:37:in `each'
	 1: from /Users/travis/build/mitsuo0114/review/lib/epubmaker/epubv2.rb:42:in `block in opf_metainfo'
/Users/travis/build/mitsuo0114/review/lib/epubmaker/epubv2.rb:42:in `escapeHTML': no implicit conversion of Date into String (TypeError)

@kmuto
Copy link
Owner

kmuto commented May 21, 2019

ありがとうございます、epubv2.rbのほうは反映しました。

travis, appveyorのほうはどうしましょうか。>@takahashim

@takahashim
Copy link
Collaborator

travisとappveyorの両方でepubcheckを動かす必要はあるんでしょうか?

@takahashim
Copy link
Collaborator

2.4-2.6のテストは取り込ませていただくので良さそうです。2.3以下はサポート終了なので落としてもいいですかね…。
https://www.ruby-lang.org/en/news/2019/03/31/support-of-ruby-2-3-has-ended/

@mitsuo0114
Copy link
Contributor Author

mitsuo0114 commented May 22, 2019

検討ありがとうございますmm

travisとappveyorの両方でepubcheckを動かす必要はあるんでしょうか?

これに関しては、OS間の差異を意識して両方に追加しています。
環境差異の必要性に関しては、私のWindows環境のうち1台で#1011 が発生したため、何らかの環境依存がありうるもの、と想定しています。

travisのほうはWindowsでのRuby実行が簡単ではないので、appveyorを使っています。

2.3以下はサポート終了なので落としてもいいですかね…。

こちら、修正しました。travisのbefore_installの部分に2.1/2.2の判定コードがあったのでここの削除も含めています。

@takahashim
Copy link
Collaborator

なるほど、ではいったんこれで取り込んで、なにかマズそうとか遅そうとかだったら再度調整することにさせてください。

@takahashim takahashim merged commit 6f4fcdc into kmuto:master May 22, 2019
kmuto added a commit that referenced this pull request May 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants