Skip to content

Skip online tests by default, replace server image with base64 data - #14070

Closed
iluuu1994 wants to merge 2 commits into
php:PHP-8.2from
iluuu1994:gh-14058
Closed

iluuu1994 wants to merge 2 commits into
php:PHP-8.2from
iluuu1994:gh-14058

Conversation

@iluuu1994

@iluuu1994 iluuu1994 commented Apr 28, 2024 •

Copy link
Copy Markdown
Member

Closes GH-14058

Apparently all of our CI already passes the --offline flag, I wonder if it should be enabled for at least one of them.

@iluuu1994
iluuu1994 requested a review from bukka as a code owner April 28, 2024 19:56

@arnaud-lb arnaud-lb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@iluuu1994 iluuu1994 closed this in cb55588 Apr 30, 2024
@iluuu1994

Copy link
Copy Markdown
Member Author

I merged this for master only, because distributions will now need to pass the --online flag to opt-into executing these tests.

@petk

petk commented May 1, 2024

Copy link
Copy Markdown
Member

Thanks @iluuu1994 The master branch is ok because there are also some existing patches out there for the sapi/fpm/status.html.in page and there patch would need to be updated for PHP-8.3, or 8.2. So less issues for the package maintainers. But for the next PHP versions, those patches need to be updated anyway.

@cmb69

cmb69 commented Sep 25, 2024

Copy link
Copy Markdown
Member

I wonder if it should be enabled for at least one of them.

Yes, that would be a good idea, see #16063.

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.

4 participants