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

Remove a hard dependency on the intl extension #289

Merged
merged 3 commits into from Oct 10, 2019
Merged

Remove a hard dependency on the intl extension #289

merged 3 commits into from Oct 10, 2019

Conversation

jakzal
Copy link
Contributor

@jakzal jakzal commented Sep 27, 2019

Fixes #288.

While #268 and #269 updated count() and isCountable() to now work with intl resources, #286 made that the whole beberlei/assert package now requires the intl extension to be installed.

However, the intl extension dependency is a soft dependency.
The package does not need intl for most of its features and can safely work without it.

Richard Quadling and others added 2 commits August 23, 2019 18:56
* v3:
  Prepare v3.2.3
Fixes #288.

While #268 and #269 updated count() and isCountable() to now work with intl resources, #286 made that the whole beberlei/assert package now requires the intl extension to be installed.

However, the intl extension dependency is a soft dependency.
The package does not need intl for most of its features and can safely work without it.
@rquadling rquadling merged commit d4207ed into beberlei:v3 Oct 10, 2019
@jakzal jakzal deleted the bugfix/optional-intl branch October 10, 2019 10:21
beberlei pushed a commit that referenced this pull request Apr 18, 2021
* Remove ext-intl as a required extension, making it a suggested extension for
use with Assertion::count(), Assertion::isCountable(), Assertion::minCount(),
and Assertion::maxCount() to operate on ResourceBundles.

* Correct @method return typehint for LazyAssertion chaining

* Restored `\Assert\Assertion::createException()` signature to 3.2.2.

Sincerely apologies to the BC break.

* Fix docblock typehint

* Prepare v3.2.4

* Remove a hard dependency on the intl extension (#289)

Fixes #288.

While #268 and #269 updated count() and isCountable() to now work with intl resources, #286 made that the whole beberlei/assert package now requires the intl extension to be installed.

However, the intl extension dependency is a soft dependency.
The package does not need intl for most of its features and can safely work without it.

* Fix docblocks for `all` to show that they are expected to receive an
array of things

* Preapare v3.2.5

* Make `Assert\Assertion::stringify()` UTF-8 safe.

* Prepare v3.2.6

* Reinstate LazyAssertion docblock

* ## 3.2.7 - 2019-12-19

### Fixes
- Reinstated the `@method` return type for `Assert\LazyAssertion` methods to show that the return type is `LazyAssertion`.

* Fixes

Co-authored-by: Richard Quadling <Richard.Quadling@DigiTickets.co.uk>
Co-authored-by: Jakub Zalas <jakub@zalas.pl>
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

2 participants