Skip to content

Multiple improvements to the library.#34

Open
Jaesin wants to merge 9 commits intodchesterton:masterfrom
Jaesin:master
Open

Multiple improvements to the library.#34
Jaesin wants to merge 9 commits intodchesterton:masterfrom
Jaesin:master

Conversation

@Jaesin
Copy link

@Jaesin Jaesin commented Aug 5, 2016

This pull request has the folowing:

  • Fixes the docblocks.
  • Adds some basic tests.
  • Adds an interface for the client.
  • Adds Travis CI which is free for open source projects.
  • Adds a getFields() (Describe lead) call,

Let me know if you need help maintaining this library.

@Jaesin
Copy link
Author

Jaesin commented Aug 12, 2016

Let me know if there are any adjustments needed for this pull request.

Test results are here: https://travis-ci.org/Jaesin/marketo-rest-api

@Jaesin
Copy link
Author

Jaesin commented Oct 11, 2016

@dchesterton I converted the API integration tests to use the built in guzzle server instead. Let me know how we can work together on getting this pull request merged.

It would be great if you could enable travis ci for this repo. Test results here: https://travis-ci.org/Jaesin/marketo-rest-api

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.

1 participant