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

Y.Log() error #1897

Closed
wants to merge 2 commits into from
Closed

Y.Log() error #1897

wants to merge 2 commits into from

Conversation

jonmak08
Copy link
Contributor

@jonmak08 jonmak08 commented Jul 3, 2014

@yahoocla
Copy link

yahoocla commented Jul 3, 2014

Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! 😄

@okuryu
Copy link
Member

okuryu commented Jul 3, 2014

@jonmak08 Thanks, looks good to me. Could you sign our CLA as above?

@jonmak08
Copy link
Contributor Author

jonmak08 commented Jul 3, 2014

Hey @okuryu,

I did sign the CLA above. Is there additional things to sign?

yahoo open source contributor license agreement

@okuryu
Copy link
Member

okuryu commented Jul 3, 2014

@jonmak08 Thank you! I'd merge this later.

@okuryu okuryu self-assigned this Jul 4, 2014
@okuryu
Copy link
Member

okuryu commented Jul 4, 2014

I have been cherry-picked into dev-master branch at 3a4bd93. It will be available in the next release.

For the future infomation, we generally work on some branches. Please see branch infomation.

@jonmak08
Copy link
Contributor Author

jonmak08 commented Jul 7, 2014

Hey @okuryu,

Thanks for the info! I'll be sure to fix the branch my changes will be sent against next time.

@jonmak08 jonmak08 deleted the y-log-error branch July 7, 2014 16:09
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