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

空のtableをエラーにする #1326

Merged
merged 3 commits into from
Jun 11, 2019
Merged

空のtableをエラーにする #1326

merged 3 commits into from
Jun 11, 2019

Conversation

kmuto
Copy link
Owner

@kmuto kmuto commented Jun 10, 2019

#1325 の対応です。
return if rows.empty? を早期に実行して開始部に至らないようにします。

@kmuto kmuto requested a review from takahashim June 10, 2019 00:24
@takahashim
Copy link
Collaborator

これですが、エラーにならなくて大丈夫なんでしょうか? 言い換えると、書き間違いではなく、空のテーブルは書くけど何も出力して欲しい、というユースケースはあるんでしょうか。

(私の方では特に困った記憶がないので、エラーでも無視でもどちらでも構わないのですが…)

@kmuto
Copy link
Owner Author

kmuto commented Jun 10, 2019

実際書き間違い以外なさそうな気はしています。
普通にエラーにしましょうか…。returnする代わりにerrorにすればいいだけですね。

@kmuto
Copy link
Owner Author

kmuto commented Jun 10, 2019

前はなんで途中でreturnしてたんだろう、記憶がさだかでない…。

@kmuto kmuto changed the title 空のtableを無視する 空のtableをエラーにする Jun 11, 2019
@kmuto kmuto merged commit e6010b1 into master Jun 11, 2019
@kmuto kmuto deleted the skip-empty-table branch June 11, 2019 00:16
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.

2 participants