Skip to content
This repository has been archived by the owner on Jan 24, 2019. It is now read-only.

Add LinkedIn provider #84

Merged
merged 1 commit into from Apr 18, 2015
Merged

Add LinkedIn provider #84

merged 1 commit into from Apr 18, 2015

Conversation

balshor
Copy link

@balshor balshor commented Apr 17, 2015

The modifications from #79 made this much cleaner than last time. Thanks for the refactoring and tests, @mbland!

@jehiah
Copy link
Member

jehiah commented Apr 18, 2015

Yay! exciting @balshor @mbland I think this is what the results of successful refactoring should look like.

@balshor would you mind squashing down to a single commit and i'll merge this in?

@balshor
Copy link
Author

balshor commented Apr 18, 2015

Done!

jehiah added a commit that referenced this pull request Apr 18, 2015
@jehiah jehiah merged commit 26170c5 into bitly:master Apr 18, 2015
@mbland
Copy link
Contributor

mbland commented Apr 18, 2015

Very cool! Glad the providers pattern proved effective and easy to follow. :-)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

None yet

3 participants