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

refactor: reduce coupling and bump to php 8 #81

Merged
merged 9 commits into from
Mar 5, 2023

Conversation

warlof
Copy link
Member

@warlof warlof commented Aug 25, 2022

remove guzzle, predis, memcache, memcached and monolog from direct dependencies of the library.
use PSR packages as replacement which allow any package compliant with the standard to be used without a need to rewrite or alter eseye library.

BREAKING CHANGES

  • logger MUST now implement Psr\Log\LoggerInterface instead of Seat\Eseye\Log\LogInterface
  • fetcher is non longer required and override to it SHOULD be applied on http_client, http_request_factory and http_stream_factory
  • default cache is now set on NullCache
  • default logger is now set on NullLogger
  • Seat\Eseye\Fetchers\GuzzleFetcher has been replaced by generic Seat\Eseye\Fetchers\Fetcher
  • minimum required PHP version is now 8.0

TODO

  • more tests
  • update documentation
  • update example

remove guzzle, predis, memcache, memcached and monolog from direct dependencies of the library.
use PSR packages as replacement which allow any package compliant with the standard to be used without a need to rewrite or alter eseye library.

**BREAKING CHANGES**

* logger MUST now implement Psr\Log\LoggerInterface instead of Seat\Eseye\Log\LogInterface
* fetcher is non longer required and override to it SHOULD be applied on http_client, http_request_factory and http_stream_factory
* default cache is now set on NullCache
* default logger is now set on NullLogger
* Seat\Eseye\Fetchers\GuzzleFetcher has been replaced by generic Seat\Eseye\Fetchers\Fetcher
* minimum required PHP version is now 8.0

**TODO**

* more tests
* update documentation
* update example
@warlof warlof changed the base branch from master to 3.x March 5, 2023 19:32
@warlof warlof marked this pull request as ready for review March 5, 2023 20:41
@warlof warlof merged commit 1f3726a into eveseat:3.x Mar 5, 2023
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

1 participant