build: adopt central Fab release pipeline - #2
Merged
Conversation
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.
Summary - Preserve the existing six migration commits and add
8d57067244aba9ca373a8b14837eedcd6250581cas the seventh normal commit. - Fix the CI checksum sidecar regex from PowerShell-expanded`r/`ntext to[\r\n], allowing ordinaryr/nin filenames while rejecting actual CR/LF. - Keep the central tool pinned tode784111d9e88a3e13205494aa56ccb12a8740f3/0.3.1and the Sample pin atb4c78058993e68a62fc9c16b673aec65ee668573. ## Commit history 1.build: adopt central Fab release pipeline2.fix: normalize Git root validation for release scripts3.fix: satisfy central Fab release validation4.fix: add test module copyright notice5.fix: validate packaged smoke artifact names6.fix: support UE 5.8 packaged runtime dependencies7.fix: validate CI artifact filenamesstyle: format runtime dependency loading## Verification - Targeted checksum regression: 4 valid filenames accepted; 3 filenames containing CR/LF rejected. - CI workflow diff is exactly one line; trigger, self-hosted Windows runner, 10-minute timeout, pins, artifact name, upload path, checksum/report/gate checks are unchanged. - UE 5.4 Verify:Total=68 Succeeded=68 SucceededWithWarnings=0 Failed=0 NotRun=0. - UE 5.8 release: tool0.3.1, repository head8d57067244aba9ca373a8b14837eedcd6250581c, 12/12 gates PASS, checksum matches. - UE 5.8 Packaged Shipping Smoke: all 9 formats pass with positive geometry/material counts, material is present in IoStore, Assimp loads from the packaged runtime directory, runtime log has no DLL load failure, memory denial and compressed texture guards pass. - No product runtime API change, Sample remote unchanged, test module not included.RuntimeAssetImport.cpphas formatting-only changes; its non-whitespace content and string literals are unchanged.Total=68 Succeeded=68 SucceededWithWarnings=0 Failed=0 NotRun=0on the new head.