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

Update codestyle, php version, tests, add interface #59

Merged
merged 8 commits into from
Jan 15, 2022

Conversation

Jeroeny
Copy link
Contributor

@Jeroeny Jeroeny commented Oct 31, 2021

This library hasn't been updated in a while. I'm not sure if that's intentional, but I've taken an attempt to modernize this library a bit. I hope an update for this library can be considered. If this is not the right approach of some changes should be left out / split in other PRs, let me know.

I've added an interface, which would be nice to have for dependency injection and mocking in tests. Updated PHPUnit and the tests. Used the last maintained PHP version as minimum and used types where possible. Applied the PSR codestyle. Added GitHub actions configuration.

code-style (PSR) check with phpcs
add interface for the client
update phpunit version and tests
use github actions
Copy link
Contributor

@andrew-demb andrew-demb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not a maintainer, but also can take a look for changes.

Overall I think PR is fine 👍
Probably major release will be required

src/Exception/ConfigurationException.php Outdated Show resolved Hide resolved
src/StatsDClient.php Outdated Show resolved Hide resolved
src/Client.php Show resolved Hide resolved
Jeroeny and others added 3 commits October 31, 2021 15:47
Co-authored-by: Andrii Dembitskyi <andrew.dembitskiy@gmail.com>
Co-authored-by: Andrii Dembitskyi <andrew.dembitskiy@gmail.com>
@Jeroeny
Copy link
Contributor Author

Jeroeny commented Nov 3, 2021

@marcqualie would you be willing to take a look?

@xvilo
Copy link

xvilo commented Dec 23, 2021

This PR looks good to me 👍

@Jeroeny
Copy link
Contributor Author

Jeroeny commented Jan 10, 2022

@colinodell I think this can be merged now?

@colinodell
Copy link
Member

I'll get it merged sometime this week (possibly this weekend). I'd also like to review the other outstanding PRs and see if we can include them too :)

composer.json Outdated Show resolved Hide resolved
@colinodell colinodell merged commit b01e4f4 into thephpleague:master Jan 15, 2022
@colinodell
Copy link
Member

Version 2.0.0 has been released! Thanks so much for your help with this :)

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.

5 participants