Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

CI Regression Test for determinism of our wasm runtimes #1372

Merged
merged 6 commits into from
Jul 20, 2020

Conversation

gnunicorn
Copy link
Contributor

analog to paritytech/substrate#6597 this builds our runtimes twice to confirm their checksums match up.

@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Jul 7, 2020
Copy link
Contributor

@s3krit s3krit left a comment

Choose a reason for hiding this comment

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

Couple of small issues, due to differences between substrate and polkadot's .gitlab-ci.yml

Co-authored-by: s3krit <pugh@s3kr.it>
@gnunicorn gnunicorn added B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. A8-mergeoncegreen and removed A0-please_review Pull request needs code review. labels Jul 20, 2020
@bkchr bkchr merged commit df6c17d into master Jul 20, 2020
@bkchr bkchr deleted the ben-deterministic-wasm branch July 20, 2020 08:56
test-deterministic-wasm:
stage: test
<<: *docker-env
except:
Copy link
Contributor

Choose a reason for hiding this comment

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

@gnunicorn I think you forgot something here, and somewhy it passed the YAML linter without having a value. When do you want to run this job?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants