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

Allow first function to return an empty array. #5247

Merged
merged 1 commit into from Sep 22, 2018

Conversation

Projects
None yet
3 participants
@felicianotech
Contributor

felicianotech commented Sep 22, 2018

Fixes #5235.

@moorereason

This comment has been minimized.

Show comment
Hide comment
@moorereason

moorereason Sep 22, 2018

Contributor

Can you write a test case for this change?

Contributor

moorereason commented Sep 22, 2018

Can you write a test case for this change?

@felicianotech

This comment has been minimized.

Show comment
Hide comment
@felicianotech

felicianotech Sep 22, 2018

Contributor

@moorereason Good point. Incoming.

Contributor

felicianotech commented Sep 22, 2018

@moorereason Good point. Incoming.

@bep bep requested a review from moorereason Sep 22, 2018

@bep bep merged commit cae07ce into gohugoio:master Sep 22, 2018

3 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
license/cla Contributor License Agreement is signed.
Details
@bep

This comment has been minimized.

Show comment
Hide comment
@bep

bep Sep 22, 2018

Member

For the future, please note the commit message guidelines.

Member

bep commented Sep 22, 2018

For the future, please note the commit message guidelines.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment