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

Fix path of the polkadot_injected docker image #4463

Merged
merged 1 commit into from
Dec 3, 2021
Merged

Fix path of the polkadot_injected docker image #4463

merged 1 commit into from
Dec 3, 2021

Conversation

chevdor
Copy link
Contributor

@chevdor chevdor commented Dec 3, 2021 •
edited
Loading

scripts/docker-> scripts/dockerfiles

This PR fixes the issues we see here

@chevdor chevdor added A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). A0-please_review Pull request needs code review. 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. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit. labels Dec 3, 2021
@chevdor chevdor requested a review from a team as a code owner December 3, 2021 16:39
@chevdor
Copy link
Contributor Author

chevdor commented Dec 3, 2021

bot merge

@paritytech-processbot
Copy link

Error:
Error: When trying to meet the "Project Owners" approval requirements: this pull request does not belong to a project defined in Process.json.

Approval by "Project Owners" is only attempted if other means defined in the criteria for merge are not satisfied first.

The following errors might have affected the outcome of this attempt:

  • 'Batch: Availability and Validity' does not match any projects in polkadot's Process.json
  • 'Batch: Codebase Restructure' does not match any projects in polkadot's Process.json

Merge failed. Check out the criteria for merge. If you're not meeting the approval count, check if the approvers are members of substrateteamleads or core-devs.

@chevdor chevdor merged commit 4468927 into master Dec 3, 2021
@chevdor chevdor deleted the wk-fix-docker-ci branch December 3, 2021 17:13
ordian added a commit that referenced this pull request Dec 3, 2021
@ordian
Merge branch 'master' into bernhardAssure_backed_candidates_are_sor
51dd597 
* master: (23 commits)
  Fix path of the polkadot_injected docker image (#4463)
  update docs on `validation_upgrade_frequency` (#4460)
  Fix path of the polkadot sha256 (#4458)
  Fix cumulus companion CI job (#4451)
  Add doc about runtime version bump (#4418)
  Bump trybuild from 1.0.52 to 1.0.53 (#4455)
  companion for #10231 (#4306)
  trivial fix (#4441)
  Rococo: ability to programatically assign slots to teams (#3943)
  Add parent header hash to log (#4421)
  bump tx versions (#4447)
  Companion for substrate/10347 (#4413)
  Improve paras runtime `BenchBuilder` api (#4318)
  Add CI team to `CODEOWNERS` file (#4350)
  XCM Benchmarks for Generic Instructions (#3940)
  Announce only on releases (#4417)
  Companion for  10379 (EnsureOneOf) (#4405)
  add pallet-babe/std (#4438)
  update Cargo.lock
  Squashed 'bridges/' changes from 407bf44a8a..1602249f0a
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@alvicsam alvicsam alvicsam approved these changes

@s3krit s3krit Awaiting requested review from s3krit

@TriplEight TriplEight Awaiting requested review from TriplEight

Assignees
No one assigned
Labels
A0-please_review Pull request needs code review. 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. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chevdor @alvicsam