Skip to content

Conversation

aliabbasrizvi
Copy link
Contributor

No description provided.

@aliabbasrizvi aliabbasrizvi requested a review from a team as a code owner July 7, 2020 17:21
@aliabbasrizvi aliabbasrizvi removed their assignment Jul 7, 2020
Copy link
Contributor

@mikeproeng37 mikeproeng37 left a comment

Choose a reason for hiding this comment

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

Wrong version in changelog :)

CHANGELOG.md Outdated
@@ -1,5 +1,17 @@
# Optimizely Python SDK Changelog

## 3.4.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Should be 3.5.0

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Already fixed. Refresh

Copy link
Contributor

@mikeproeng37 mikeproeng37 left a comment

Choose a reason for hiding this comment

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

lgtm

@coveralls
Copy link

coveralls commented Jul 7, 2020

Coverage Status

Coverage remained the same at 97.096% when pulling 4d0a697 on ali/prep_for_release into f9484bd on master.

@aliabbasrizvi aliabbasrizvi force-pushed the ali/prep_for_release branch from b76e799 to 4d0a697 Compare July 8, 2020 20:18
@aliabbasrizvi aliabbasrizvi removed their assignment Jul 8, 2020
@aliabbasrizvi aliabbasrizvi merged commit 3e0fa0a into master Jul 8, 2020
@aliabbasrizvi aliabbasrizvi deleted the ali/prep_for_release branch July 8, 2020 21:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants