Skip to content

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