Skip to content

Update requirement: redis-py 3#218

Closed
akachanov wants to merge 1 commit into
snowplow:masterfrom
akachanov:redis_py_3
Closed

Update requirement: redis-py 3#218
akachanov wants to merge 1 commit into
snowplow:masterfrom
akachanov:redis_py_3

Conversation

@akachanov

Copy link
Copy Markdown
Contributor

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 80.482% when pulling c4cad40 on akachanov:redis_py_3 into 4550310 on snowplow:master.

1 similar comment
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 80.482% when pulling c4cad40 on akachanov:redis_py_3 into 4550310 on snowplow:master.

@snowplowcla

Copy link
Copy Markdown

Thanks for your pull request. Is this your first contribution to a Snowplow open source project? Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://github.com/snowplow/snowplow/wiki/CLA to learn more and sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

@akachanov

Copy link
Copy Markdown
Contributor Author

I signed it

@snowplowcla

Copy link
Copy Markdown

Confirmed! akachanov has signed the Individual Contributor License Agreement. Thanks so much

@BenFradet Ben Fradet (BenFradet) left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM 👍, assigning to Michael Hadam (@mhadam) to have a look

@oguzhanunlu

Copy link
Copy Markdown
Member

Thanks for the contribution akachanov ! It is cherry-picked to the release branch at #224 .

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.

6 participants