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

Implement dependent check for Cumulus #4017

Merged
merged 1 commit into from
Oct 14, 2021
Merged

Conversation

joao-paulo-parity
Copy link
Contributor

@joao-paulo-parity joao-paulo-parity commented Oct 5, 2021

❗ After this PR is merged, the required checks in the Github Repository Settings should be updated:

  • continuous-integration/gitlab-check-dependent-cumulus was added (should be required)

For more context see https://github.com/paritytech/ci_cd/issues/227

@joao-paulo-parity joao-paulo-parity marked this pull request as ready for review October 5, 2021 14:19
@joao-paulo-parity joao-paulo-parity added A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). 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. labels Oct 5, 2021
@bkchr bkchr requested a review from TriplEight October 8, 2021 11:34
.gitlab-ci.yml Outdated Show resolved Hide resolved
.gitlab-ci.yml Outdated Show resolved Hide resolved
.gitlab-ci.yml Outdated Show resolved Hide resolved
.gitlab-ci.yml Outdated Show resolved Hide resolved
.gitlab-ci.yml Show resolved Hide resolved
@joao-paulo-parity joao-paulo-parity force-pushed the jp/dependent-check branch 2 times, most recently from 6e8b678 to ce2fbd8 Compare October 14, 2021 12:16
.gitlab-ci.yml Outdated
Comment on lines 280 to 281
--depth=1
"--branch=$PIPELINE_SCRIPTS_TAG"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
--depth=1
"--branch=$PIPELINE_SCRIPTS_TAG"
--depth=1
"--branch=$PIPELINE_SCRIPTS_TAG"

.gitlab-ci.yml Outdated
- git clone
--depth=1
"--branch=$PIPELINE_SCRIPTS_TAG"
https://github.com/paritytech/pipeline-scripts
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
https://github.com/paritytech/pipeline-scripts
https://github.com/paritytech/pipeline-scripts

@TriplEight TriplEight merged commit 9866a9c into master Oct 14, 2021
@TriplEight TriplEight deleted the jp/dependent-check branch October 14, 2021 12:25
ordian added a commit that referenced this pull request Oct 14, 2021
* master: (40 commits)
  use pipeline-scripts for dependent projects script (#4017)
  companion for substrate/10013 (#4078)
  fix restart issue of staking miner (#4073)
  use `WEIGHT_PER_SECOND` (#4077)
  Switch to compressed runtimes (#4061)
  Create a README for XCMv1 detailing notable changes (#4058)
  Bump proc-macro2 from 1.0.29 to 1.0.30 (#4071)
  Move runtime to with_transaction in order to Rollback state changes (#4067)
  Update wasmtime config to match new version in substrate (#4063)
  Fix XCM Weights on Westend (#4066)
  Dispatchable XCMs should translate to v0 (#4062)
  Add xcm fuzzer (#3869)
  Bump trybuild from 1.0.49 to 1.0.50 (#4060)
  Companion PR for ss58 crate (#3953)
  overseer: simplify debugging some more (#4053)
  Bump thiserror from 1.0.29 to 1.0.30 (#4048)
  Bump strum from 0.21.0 to 0.22.0 (#4047)
  Add para pallets to polkadot benchmarks (#4045)
  Fixes broken docker link in docker descriptiob (#4043)
  Bump trybuild from 1.0.48 to 1.0.49 (#4039)
  ...
ordian added a commit that referenced this pull request Oct 14, 2021
…lkadot

* master:
  use pipeline-scripts for dependent projects script (#4017)
  companion for substrate/10013 (#4078)
  fix restart issue of staking miner (#4073)
  use `WEIGHT_PER_SECOND` (#4077)
  Switch to compressed runtimes (#4061)
  Create a README for XCMv1 detailing notable changes (#4058)
  Bump proc-macro2 from 1.0.29 to 1.0.30 (#4071)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). 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.

3 participants