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

Spotify authentication #1226

Merged
merged 1 commit into from
Apr 5, 2016
Merged

Spotify authentication #1226

merged 1 commit into from
Apr 5, 2016

Conversation

1nput0utput
Copy link

No description provided.

@codecov-io
Copy link

Current coverage is 92.71%

Merging #1226 into master will decrease coverage by -0.29% as of ee0120c

@@            master   #1226   diff @@
======================================
  Files           83      84     +1
  Stmts         5287    5312    +25
  Branches       967     970     +3
  Methods          0       0       
======================================
+ Hit           4917    4925     +8
  Partial         10      10       
- Missed         360     377    +17

Review entire Coverage Diff as of ee0120c

Powered by Codecov. Updated on successful CI builds.

@flovilmart
Copy link
Contributor

@1nput0utput thanks for the PR, we're taking some time to review that one as we are currently pondering moving the oauth authenticators to a separate repository

@facebook-github-bot
Copy link

@1nput0utput updated the pull request.

1 similar comment
@facebook-github-bot
Copy link

@1nput0utput updated the pull request.

@gfosco
Copy link
Contributor

gfosco commented Apr 5, 2016

Thanks for the feature, @1nput0utput 👍

We'll merge this in to master now. In the near future, we may move all of these to separate modules.

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.

5 participants