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

gcs: Support empty filters. #1844

Merged
merged 1 commit into from
Aug 20, 2019
Merged

Conversation

davecgh
Copy link
Member

@davecgh davecgh commented Aug 19, 2019

This requires PR #1843.

This adds support for empty filters versus being an error along with a full set of tests to ensure the empty filter works as intended.

It is part of the onging process to cleanup and improve the gcs module to the quality level required by consensus code for ultimate inclusion in header commitments.

@davecgh davecgh added this to the 1.5.0 milestone Aug 19, 2019
@davecgh davecgh force-pushed the gcs_allow_empty_filters branch 2 times, most recently from a0b8422 to 87bf1ab Compare August 19, 2019 17:50
Copy link
Member

@dnldd dnldd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

This adds support for empty filters versus being an error along with a
full set of tests to ensure the empty filter works as intended.

It is part of the onging process to cleanup and improve the gcs module
to the quality level required by consensus code for ultimate inclusion
in header commitments.
@davecgh davecgh force-pushed the gcs_allow_empty_filters branch from 87bf1ab to 468f328 Compare August 20, 2019 14:13
@davecgh davecgh merged commit 468f328 into decred:master Aug 20, 2019
@davecgh davecgh deleted the gcs_allow_empty_filters branch August 20, 2019 14:32
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.

5 participants