Skip to content

Conversation

@TheOtterlord
Copy link
Member

Changes

Fix build errors being ignored when using concurrency, by using Promise.all instead of Promise.allSettled

Testing

Added a test fixture that throws an error when built.

Docs

N/A (expected behaviour)

@changeset-bot
Copy link

changeset-bot bot commented Apr 6, 2025

🦋 Changeset detected

Latest commit: ee63074

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added the pkg: astro Related to the core `astro` package (scope) label Apr 6, 2025
@TheOtterlord TheOtterlord linked an issue Apr 6, 2025 that may be closed by this pull request
1 task
@codspeed-hq
Copy link

codspeed-hq bot commented Apr 6, 2025

CodSpeed Performance Report

Merging #13566 will not alter performance

Comparing otter/concurrency (ee63074) with main (e588527)

Summary

✅ 6 untouched benchmarks

@florian-lefebvre florian-lefebvre merged commit 0489d8f into main May 13, 2025
17 checks passed
@florian-lefebvre florian-lefebvre deleted the otter/concurrency branch May 13, 2025 19:11
@astrobot-houston astrobot-houston mentioned this pull request May 13, 2025
openscript pushed a commit to openscript/astro that referenced this pull request Sep 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg: astro Related to the core `astro` package (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Build errors are ignored when building with concurrency

2 participants