Skip to content

Conversation

@silasjoisten
Copy link
Collaborator

@silasjoisten silasjoisten commented May 15, 2025

closes #653

@mvhirsch can you please test this?

@silasjoisten silasjoisten requested review from OskarStark and shish May 15, 2025 05:46
@silasjoisten silasjoisten self-assigned this May 15, 2025
@silasjoisten silasjoisten changed the title Adds support for ftp_raw on PHP 8.1 feat(PHP 8.1): support for ftp_raw May 15, 2025
@silasjoisten silasjoisten force-pushed the feature/ftp_raw-php-8.1 branch from 283f7b1 to 4a732ac Compare May 15, 2025 06:07
@silasjoisten
Copy link
Collaborator Author

@cedric-anne WDYT?

@cedric-anne
Copy link
Contributor

@cedric-anne WDYT?

It could also be detected using a specific pattern in the generator/config/detectErrorType.php script to detect the retrun value that was documented prior to php/doc-en@0773339

@silasjoisten
Copy link
Collaborator Author

@cedric-anne aparently not. because again the version is pinned to a commit means it wont recognize the changes. the array already contains that very pattern

@silasjoisten silasjoisten merged commit 55838ec into master May 16, 2025
10 checks passed
@silasjoisten silasjoisten deleted the feature/ftp_raw-php-8.1 branch May 16, 2025 06:15
@mvhirsch
Copy link
Contributor

mvhirsch commented May 19, 2025

@mvhirsch can you please test this?

Unfortunately not. I'm not using this function anymore in the project in question.

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.

Missing Function: ftp_raw for PHP 8.1

4 participants