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

Release 3.0.0 #27

Merged
merged 16 commits into from
Mar 25, 2021
Merged

Release 3.0.0 #27

merged 16 commits into from
Mar 25, 2021

Conversation

mtompset
Copy link
Contributor

Change

  • Force minimum PHP 7.2 requirement

Fix

  • Update testing container to use PHP 7.3
  • Update PHPUnit to 9.x
  • Code cleanup with transition to PHP 7.3
  • Refactor Psr3Yii2Logger::interpolate to only return string

Add

  • Psr3Yii2Logger tests

mtompset and others added 16 commits March 24, 2021 16:07
- return void is the new return null
- imported InvalidArguementException
- Simplified case statements
- Fixed PHPDoc
- Did array checking to use new interpolateArray() vs. interpolate()
- Removed isArraySupported, since only Psr3Yii2Logger supports arrays
- Removed interpolateArray() call in favour of an inline array_merge()
- Strongly type interpolateString()
- Move EMERGENCY log level case
- remove isArraySupported
- replace call to interpolateArray() with array_merge()
…-improvements

Feature/php7.3 and other improvements
@mtompset mtompset requested a review from forevermatt March 25, 2021 15:44
@mtompset mtompset self-assigned this Mar 25, 2021
@mtompset mtompset merged commit c12e60f into master Mar 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants