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

Implemented HttpAdapter with guzzle #244

Closed
wants to merge 2 commits into from
Closed

Implemented HttpAdapter with guzzle #244

wants to merge 2 commits into from

Conversation

GrahamCampbell
Copy link
Member

Replaces #178.

@GrahamCampbell
Copy link
Member Author

Dang. This needs rebasing again...

Herberto Graca and others added 2 commits September 2, 2014 21:50
@GrahamCampbell
Copy link
Member Author

@FrenkyNet Is this good to go then? You said on the original pull you'd be happy to merge it an tweak it yourself after it was rebased, and here it is rebased, and I've added readme docs. :)

@frankdejonge
Copy link
Member

@GrahamCampbell do you know if it's possible to create a PR for a feature branch? This way I can work on it before merging it into master.

@staabm
Copy link

staabm commented Sep 4, 2014

@FrenkyNet you can just open a PR against GrahamCampbell:guzzle on @GrahamCampbell fork

@GrahamCampbell
Copy link
Member Author

If you like, you could always merge my pull request yourself using git to another branch on your repo?

Or as @staabm said, you can always send pull requests to my fork.

@frankdejonge
Copy link
Member

Merged this into a feature branch.

@GrahamCampbell GrahamCampbell deleted the guzzle branch September 6, 2014 10:11
@GrahamCampbell GrahamCampbell mentioned this pull request Jul 9, 2015
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.

None yet

3 participants