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

Move providers to new namespace #604

Merged
merged 11 commits into from May 13, 2017

Conversation

Projects
None yet
1 participant
@Nyholm
Member

Nyholm commented Apr 28, 2017

This is a first PR for #578

I've just done the Chain provider the moment. All other providers will follow the same pattern.

TODO

  • Move the rest of the providers
  • Update change log
  • Make sure travis runs the tests

@Nyholm Nyholm added this to the 4.0.0 milestone Apr 28, 2017

@willdurand willdurand self-requested a review Apr 28, 2017

Nyholm added some commits May 12, 2017

Namespace local (#1)
* fix

* bugfix

* Make sure we run tests for common

* tests

* debug

* Added some config

* Updated path to phpunit

* debug

* debub

* added phpunit file

* minor

* Fixed to get phpunit-bridge to work

* Do not use simple-phpunit

* Fixed broken tests

* Fixed broken tests

* Use curl-client and guzzlehttp/psr7: ^1.4

* Remove Guzzle

* Require latest version of CurlClient
Namespace patch 1 (#2)
* Make sure tests run

* Use Curl

* Renamed cached files
@Nyholm

This comment has been minimized.

Member

Nyholm commented May 13, 2017

Im done with this PR. Deps low will fail because it is using code in master.

@Nyholm Nyholm merged commit 3180c7b into geocoder-php:master May 13, 2017

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/styleci/pr The StyleCI analysis has passed
Details

@Nyholm Nyholm deleted the Nyholm:namespace branch May 13, 2017

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