Skip to content

Conversation

guillaumelecerf
Copy link
Contributor

No description provided.

@guillaumelecerf guillaumelecerf marked this pull request as draft November 3, 2021 15:29
@guillaumelecerf guillaumelecerf force-pushed the feature/handle-empty-resultset-as-zero branch from 4391a2c to c627402 Compare November 3, 2021 15:49
@guillaumelecerf guillaumelecerf marked this pull request as ready for review November 3, 2021 15:52
@guillaumelecerf guillaumelecerf force-pushed the feature/handle-empty-resultset-as-zero branch 4 times, most recently from 209ccfa to d6ecad2 Compare November 3, 2021 16:34
@raffis
Copy link
Owner

raffis commented Nov 4, 2021

Thanks for the contribution @guillaumelecerf , can you add a test case for this? should be fairly simple 👍🏻

@guillaumelecerf guillaumelecerf force-pushed the feature/handle-empty-resultset-as-zero branch 2 times, most recently from c5becca to 5bdb478 Compare November 4, 2021 09:53
@guillaumelecerf guillaumelecerf changed the title Add an option to handle empty result set as 0 Add an option to override value for empty result set Nov 4, 2021
@guillaumelecerf guillaumelecerf force-pushed the feature/handle-empty-resultset-as-zero branch 2 times, most recently from a3f1935 to 7046eda Compare November 4, 2021 11:08
@guillaumelecerf
Copy link
Contributor Author

@raffis : here is a new version allowing to override empty result set with an arbitrary value. I also added a simple unittest.

@guillaumelecerf guillaumelecerf force-pushed the feature/handle-empty-resultset-as-zero branch 3 times, most recently from 1dcfaec to 3e58509 Compare November 4, 2021 11:41
@raffis
Copy link
Owner

raffis commented Nov 4, 2021

Thanks for the contribution 🎉

@raffis raffis merged commit d777ee2 into raffis:master Nov 4, 2021
@guillaumelecerf guillaumelecerf deleted the feature/handle-empty-resultset-as-zero branch November 4, 2021 15:29
@guillaumelecerf
Copy link
Contributor Author

@raffis : thanks for the merge. Would it be possible for you to create a beta version (available on DockerHub) ?

@raffis
Copy link
Owner

raffis commented Nov 4, 2021

@raffis : thanks for the merge. Would it be possible for you to create a beta version (available on DockerHub) ?

Yes will release any minute.

@raffis
Copy link
Owner

raffis commented Nov 4, 2021

@raffis : thanks for the merge. Would it be possible for you to create a beta version (available on DockerHub) ?

Yes will release any minute.

You can test it using v1.0.0-beta8

@guillaumelecerf
Copy link
Contributor Author

Thanks a lot, it works as expected 👍

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