Skip to content

PHP 8.4: Avoid deprecated notice - #118

Open
gharlan wants to merge 1 commit into
pascaldevink:masterfrom
gharlan:php84-deprecated
Open

PHP 8.4: Avoid deprecated notice#118
gharlan wants to merge 1 commit into
pascaldevink:masterfrom
gharlan:php84-deprecated

Conversation

@gharlan

@gharlan gharlan commented Dec 5, 2024

Copy link
Copy Markdown
Contributor

Deprecated: PascalDeVink\ShortUuid\ShortUuid::__construct(): Implicitly marking parameter $alphabet as nullable is deprecated, the explicit nullable type must be used instead in vendor/pascaldevink/shortuuid/src/ShortUuid.php on line 34

@TomBrouws

Copy link
Copy Markdown

@pascaldevink we're running into this notice when upgrading to PHP 8.4, any chance this can get merged? I can send an MR to add 8.4 support if you can commit to a release afterwards 🙏

@gharlan

gharlan commented Jul 8, 2025

Copy link
Copy Markdown
Contributor Author

@pascaldevink Since I know that maintaining open source software can sometimes be a pain and you don't always have time for it, I don't want to put any pressure on you.

But since the last commit was over 2 years ago, I would like to ask what your plans are with this package?
Do you plan to continue maintaining it and creating new releases when you find some time? Then I will wait patiently.
Or do you actually not want to continue maintaining it? Then we should consider whether someone should create a fork. Or would you like to add someone here in the repo as a contributor with merge+release permissions?

@sandervanhooft

Copy link
Copy Markdown

Hey @pascaldevink ,

Hope you're well. Let me know if you need any help here.

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.

3 participants