Skip to content

Conversation

davidmorgan
Copy link
Contributor

@davidmorgan davidmorgan commented Sep 25, 2025

Start testing on MacOS.

Fix a bug in build_daemon that this uncovers. The system temp folder on Mac is symlinked, and this was causing the build daemon e2e test to fail because of the test and the build daemon treating the symlink-unresolved workspace as different from the symlink-resolved workspace.

There is some flakiness in the daemon end to end test related to unexpected file change events, "add" of an existing file, but this code will change soon so live with the flakiness for now.

Copy link

github-actions bot commented Sep 25, 2025

PR Health

Changelog Entry ✔️
Package Changed Files

Changes to files need to be accounted for in their respective changelogs.

This check can be disabled by tagging the PR with skip-changelog-check.

@davidmorgan davidmorgan force-pushed the test-on-mac branch 13 times, most recently from 4cd4f23 to 657202a Compare September 26, 2025 09:05
@davidmorgan davidmorgan marked this pull request as ready for review September 26, 2025 09:32
@davidmorgan davidmorgan merged commit 07b5911 into dart-lang:master Sep 26, 2025
35 of 37 checks passed
@davidmorgan davidmorgan deleted the test-on-mac branch September 26, 2025 09:55
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.

2 participants