Skip to content

Fix parsing of v1 file #31

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

Merged
merged 1 commit into from
Oct 3, 2016
Merged

Fix parsing of v1 file #31

merged 1 commit into from
Oct 3, 2016

Conversation

aliabbasrizvi
Copy link
Contributor

@optimizely/fullstack-devs

@aliabbasrizvi
Copy link
Contributor Author

The failures are due to dependency install failures

@coveralls
Copy link

coveralls commented Oct 3, 2016

Coverage Status

Coverage increased (+0.003%) to 97.626% when pulling a154832 on aliabbasrizvi/fix_for_v1 into e987130 on devel.

@aliabbasrizvi
Copy link
Contributor Author

Re-ran tests and they went through fine.

@vraja2
Copy link

vraja2 commented Oct 3, 2016

lgtm

@aliabbasrizvi aliabbasrizvi merged commit 92cf30d into devel Oct 3, 2016
@aliabbasrizvi aliabbasrizvi deleted the aliabbasrizvi/fix_for_v1 branch October 3, 2016 16:58
aliabbasrizvi added a commit that referenced this pull request Oct 3, 2016
* Changing to use objects (#21)

* Moving audiences to object (#22)

* Creating object for groups (#23)

* Moving variations to use objects (#24)

* Updating whitelisting check to precede audience check (#25)

* Updating contribution doc to use CLA (#26)

* Better exception handling in the SDK (#27)

* fix developer docs link (#28)

* Fixing logger (#29)

* Handle exceptions in event dispatcher (#30)

* Fixing parsing of v1 file (#31)

* Bump to 1.0.0
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.

3 participants