Repository navigation
🔀 MAIN - release v2.26.0 - #5
Merged
Merged
Conversation
…saction each Every statement used to commit on its own, so a fresh database paid one fsync per CREATE, ALTER and INSERT: ~500 ms per runtime, 80% of the start-up cost of every server test, which builds hundreds of them. One transaction brings it to ~35 ms and the node:test suite from 155 s to 62 s locally. It also makes an upgrade that fails half way roll back to the previous schema instead of leaving it half applied, which the new test pins.
…emulation The builder's output is plain JavaScript, identical on every architecture, so the arm64 leg now copies it from a native build instead of running npm ci, tsc and Vite under QEMU, which was most of the release's 310 s Light build.
…te check Static checks, server tests, dashboard tests and build + visual each get a runner, so a run lasts as long as its slowest job instead of the sum. The aggregate job keeps the name `verify`, which main's ruleset requires and the release now reads. coverage is split into coverage:server and coverage:web so each job runs one suite.
…f re-running tests
…t ship time_zone, resolved_at and shortlink are absent from Recurly's answer; the adapter never reads the first and last and falls back to updated_at without resolved_at, so their absence is not drift. Closes #4
The mode is state, set on render, while data-theme is written by useTheme's effect a commit later. Checked bare right after the wait, the attribute assertion raced that effect and failed under a loaded Vitest shard.
…to the merge Each shard records coverage without judging it, since half the files are always under the floor; dashboard-coverage merges both blob reports and holds the whole to the same thresholds coverage:web uses locally.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Merge dev into main for v2.26.0, without the excluded local paths.