Skip to content

fixo #2702

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
Jun 8, 2017
Merged

fixo #2702

merged 1 commit into from
Jun 8, 2017

Conversation

ferhtaydn
Copy link
Contributor

convert from procedure to function :)

Copy link
Member

@dottybot dottybot left a comment

Choose a reason for hiding this comment

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

Hello, and thank you for opening this PR! 🎉

All contributors have signed the CLA, thank you! ❤️

Have an awesome day! ☀️

convert from procedure to function :)
@ferhtaydn
Copy link
Contributor Author

hey @felixmulder, I don't know why it is asking CLA to me. Some small fixes to the documentation.

@felixmulder
Copy link
Contributor

felixmulder commented Jun 7, 2017

@ferhtaydn - we legally cannot accept your code (in this repo) if you don't sign the CLA here: lightbend.com/contribute/cla/scala

We're working on something to resolve that, but in the meantime please sign :)

@ferhtaydn
Copy link
Contributor Author

ferhtaydn commented Jun 7, 2017

@felixmulder I has signed it long time ago but it was for scala repo if I don't remember wrong. Anyway, I signed the CLA again. fyi. :)

@felixmulder
Copy link
Contributor

@dotty-bot: recheck CLA please ❤️

Copy link
Contributor

@felixmulder felixmulder left a comment

Choose a reason for hiding this comment

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

All green (=

Thanks for putting up with the hassle @ferhtaydn!

@felixmulder felixmulder merged commit b89e00a into scala:master Jun 8, 2017
@ferhtaydn
Copy link
Contributor Author

:) Thank you @felixmulder 💯

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