Skip to content

Conversation

@ruudk
Copy link
Contributor

@ruudk ruudk commented Jan 14, 2025

This fixes the following error:

PHP Deprecated: AmplitudeExperiment\Amplitude\Amplitude::__construct(): Implicitly marking parameter $config as nullable is deprecated, the explicit nullable type must be used instead

ruudk added 2 commits January 14, 2025 09:32
This fixes the following error:

> PHP Deprecated:  AmplitudeExperiment\Amplitude\Amplitude::__construct(): Implicitly marking parameter $config as nullable is deprecated, the explicit nullable type must be used instead
@ruudk
Copy link
Contributor Author

ruudk commented Jan 14, 2025

@tyiuhc Please have a look. If merged, please tag a new release.

@tyiuhc tyiuhc changed the title PHP 8.4 fix: use explicit nullable type for AmplitudeConfig Jan 14, 2025
@tyiuhc tyiuhc merged commit c5a91cd into amplitude:main Jan 14, 2025
7 of 8 checks passed
github-actions bot pushed a commit that referenced this pull request Jan 15, 2025
## [1.2.2](1.2.1...1.2.2) (2025-01-15)

### Bug Fixes

* use explicit nullable type for AmplitudeConfig ([#32](#32)) ([c5a91cd](c5a91cd))
@github-actions
Copy link

🎉 This PR is included in version 1.2.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@tyiuhc
Copy link
Collaborator

tyiuhc commented Jan 15, 2025

Hi @ruudk, thank you for your contribution on this issue! This fix is now released in v1.2.2.

@ruudk ruudk deleted the ruudk/2025/1/php84 branch January 15, 2025 07:27
@ruudk ruudk restored the ruudk/2025/1/php84 branch January 15, 2025 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants