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

Fix: return err if lines is empty #3

Merged
merged 2 commits into from
Jun 8, 2020
Merged

Fix: return err if lines is empty #3

merged 2 commits into from
Jun 8, 2020

Conversation

zu1k
Copy link
Contributor

@zu1k zu1k commented Jun 8, 2020

统计信息为空时抛出错误防止自动删除gist文件

未经测试

@YouEclipse
Copy link
Owner

YouEclipse commented Jun 8, 2020

感谢~ CI过了,我先merge了.
不过我倾向于把这个判断逻辑放在main函数,判断lines 的长度是否为0.

@YouEclipse YouEclipse merged commit 77dbcf3 into YouEclipse:master Jun 8, 2020
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.

None yet

2 participants