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

Add Google::getScopeInline(). #42

Merged
merged 1 commit into from
Apr 19, 2017
Merged

Conversation

ADmad
Copy link
Contributor

@ADmad ADmad commented Apr 9, 2017

Hey!

Type: enhancement

Link to issue:

In raising this pull request, I confirm the following (please check boxes):

  • I have read and understood the Contributing Guidelines.
  • I have checked that another pull request for this purpose does not exist.
  • I wrote some tests for this PR.

Small description of change:

This allows specifying each scope as a separate array value in config
instead of an array with single space separated string.

Thanks 😺

This allows specifying each scope as a separate array value in config
instead of an array with single space separated string.
@ovr
Copy link
Member

ovr commented Apr 12, 2017

Restarting tests

@ovr ovr closed this Apr 12, 2017
@ovr ovr reopened this Apr 12, 2017
@ADmad
Copy link
Contributor Author

ADmad commented Apr 12, 2017

Any particular reason you have used --prefer-source instead of --prefer-dist for composer install in travis.yml? It's slows down the build considerably :)

@ADmad
Copy link
Contributor Author

ADmad commented Apr 16, 2017

Ping, I think this is good to merge :)

@ovr
Copy link
Member

ovr commented Apr 17, 2017

My macbook is broken (waiting from service center). I will try to find another pc today to merge it and test.

@ADmad
Copy link
Contributor Author

ADmad commented Apr 17, 2017

Sounds like 1st world problem :)

@ovr
Copy link
Member

ovr commented Apr 18, 2017

I am sorry for delay. Still w8ting for my macbook :( They promise that I will get it tomorrow...
Life without macbook is not a life :(

@ovr ovr merged commit e0e3308 into SocialConnect:master Apr 19, 2017
@ovr
Copy link
Member

ovr commented Apr 19, 2017

LGTM, Thank you for patch, you are welcome!
🍻

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.

2 participants