Skip to content

Conversation

@Uzlopak
Copy link
Contributor

@Uzlopak Uzlopak commented Sep 7, 2025

Consuming the bodies ensures that the tests will not fail. Plus removing tspl in encoding.js as it seems to make issues...

This relates to...

Rationale

Changes

Features

Bug Fixes

Breaking Changes and Deprecations

Status

@Uzlopak Uzlopak force-pushed the warmup-macos-encoding branch 2 times, most recently from 9b8476b to 04081e9 Compare September 10, 2025 09:14
@Uzlopak Uzlopak requested a review from Copilot September 11, 2025 10:43

This comment was marked as resolved.

@Uzlopak Uzlopak force-pushed the warmup-macos-encoding branch from b3a6a92 to 234c00f Compare September 12, 2025 23:38
@Uzlopak Uzlopak force-pushed the warmup-macos-encoding branch from e665be5 to fb9e3bf Compare September 13, 2025 00:16
@Uzlopak Uzlopak requested a review from Copilot September 13, 2025 00:19

This comment was marked as outdated.

@Uzlopak Uzlopak force-pushed the warmup-macos-encoding branch from f9a8dd1 to 33f7fb4 Compare September 13, 2025 00:37
@Uzlopak Uzlopak requested a review from Copilot September 13, 2025 00:41

This comment was marked as outdated.

@Uzlopak Uzlopak requested a review from Copilot September 13, 2025 00:44

This comment was marked as outdated.

@Uzlopak Uzlopak requested a review from Copilot September 13, 2025 00:48

This comment was marked as outdated.

@Uzlopak Uzlopak force-pushed the warmup-macos-encoding branch from c3803fe to f4cc9a5 Compare September 13, 2025 00:53
@Uzlopak Uzlopak changed the title test: warmup for macos and node 20 test: consume bodies of fetch responses to fix failing macos 20 ci Sep 13, 2025
@Uzlopak Uzlopak requested a review from Copilot September 13, 2025 06:12

This comment was marked as outdated.

@Uzlopak
Copy link
Contributor Author

Uzlopak commented Sep 13, 2025

@metcoder95
@mcollina
@KhafraDev

Can one of you please approve? I think this time it should fix those files finally...

Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@Uzlopak Uzlopak merged commit c4ede11 into main Sep 13, 2025
67 checks passed
@github-actions github-actions bot mentioned this pull request Jan 5, 2026
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