Skip to content

Supply core library for required backends#368

Merged
tjpalmer merged 2 commits intomainfrom
supply-required-backends
Mar 5, 2026
Merged

Supply core library for required backends#368
tjpalmer merged 2 commits intomainfrom
supply-required-backends

Conversation

@tjpalmer
Copy link
Contributor

@tjpalmer tjpalmer commented Mar 4, 2026

  • This change will be explicitly tested in an external project once this pr goes in

tjpalmer added 2 commits March 3, 2026 18:33
Signed-off-by: Tom <tom@temper.systems>
Signed-off-by: Tom <tom@temper.systems>
@tjpalmer tjpalmer marked this pull request as ready for review March 4, 2026 15:48
}

@Test
@Timeout(JAVA_TIMEOUT_SECONDS) // Just keep seeing this one specifically time out for some reason.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I got tired of this timing out in ci.

val flatBackendIds = backendOrganization.backendBuckets.flatten()
supplyCoreLibrary(harness.outputFileSystem, flatBackendIds, cancelGroup, harness.cliConsole)

build.beforeStartTranslation?.await()
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This just moved down to after the backend organization instead of before.

Copy link
Contributor

Choose a reason for hiding this comment

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

Sometimes I think one test just runs early and so gets debited all the module loading and cache filling that's shared with subsequent test cases.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sometimes I think one test just runs early and so gets debited all the module loading and cache filling that's shared with subsequent test cases.

Yeah, could be. If we start seeing another one be slow, we could just give extra time to the whole class maybe.

@tjpalmer tjpalmer merged commit 6cbfc29 into main Mar 5, 2026
2 checks passed
@tjpalmer tjpalmer deleted the supply-required-backends branch March 5, 2026 20:35
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