Skip to content

Promote warnings to exceptions in ext/ftp #6054

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

Closed
wants to merge 3 commits into from

Conversation

kocsismate
Copy link
Member

No description provided.

@nikic
Copy link
Member

nikic commented Aug 31, 2020

Test failures look legit.

@nikic
Copy link
Member

nikic commented Aug 31, 2020

Still failing :)

@kocsismate
Copy link
Member Author

Noo way :( Sorry for the noise. it should be good after my newest commit.

Copy link
Member

@nikic nikic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be nice to get rid of the trailing . in the other FTP error messages as well...

@kocsismate
Copy link
Member Author

kocsismate commented Aug 31, 2020

It would be nice to get rid of the trailing . in the other FTP error messages as well...

@nikic Yeah, I noticed them, but then I realized that this message is only defined in the tests (ext/ftp/tests/server.inc)

@php-pulls php-pulls closed this in 1410cd6 Aug 31, 2020
@kocsismate kocsismate deleted the ftp-warning branch August 31, 2020 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants