This repository was archived by the owner on May 21, 2024. It is now read-only.
ci: add deterministic build option#134
Merged
Merged
Conversation
evilrobot-01
force-pushed
the
frank/feat-deterministic-build
branch
3 times, most recently
from
db7f224 to
6177e46
Compare
evilrobot-01
force-pushed
the
frank/feat-deterministic-build
branch
from
6177e46 to
0b48242
Compare
Contributor
Author
|
Sample run at https://github.com/paritytech/trappist/actions/runs/4286565726. Summary step shows full output, resulting artifacts are wasm/json as .zip. |
evilrobot-01
force-pushed
the
frank/feat-deterministic-build
branch
2 times, most recently
from
91af055 to
6b5d966
Compare
evilrobot-01
force-pushed
the
frank/feat-deterministic-build
branch
from
6b5d966 to
d30d197
Compare
Contributor
|
Awesome contribution @evilrobot-01. I think this might come in handy at some point ... 🤓️ |
Contributor
|
@stiiifff & @evilrobot-01 is there any reason why we didn't merge this PR yet? this change can be useful for #138 also you can merge this PR to this preparation branch: https://github.com/paritytech/trappist/tree/hb-deploy-preparation |
Contributor
Author
|
None from my side, guess perhaps as it wasn't a priority based on other things going on at the time. I think it was just left for the future as and when required. |
12 tasks
stiiifff
approved these changes
Apr 5, 2023
Contributor
|
LGTM |
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
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.
Quick PR based on https://docs.substrate.io/build/build-a-deterministic-runtime, adding an optional workflow which performs a deterministic build using Substrate Runtime Toolbox:
srtool.Workflows set as
workflow_dispatch, so will only run on manual trigger.