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

Companion for substrate #13159#6620

Merged
bkchr merged 4 commits intoparitytech:masterfrom
girazoki:girazoki-companion-set-members-origin
Feb 13, 2023
Merged

Companion for substrate #13159#6620
bkchr merged 4 commits intoparitytech:masterfrom
girazoki:girazoki-companion-set-members-origin

Conversation

@girazoki
Copy link
Copy Markdown
Contributor

@girazoki girazoki commented Jan 24, 2023 •
edited
Loading

Companion for substrate #13159: paritytech/substrate#13159

Companion for cumulus: paritytech/cumulus#2136

@paritytech-ci paritytech-ci requested review from a team January 24, 2023 11:50
@girazoki girazoki mentioned this pull request Jan 24, 2023
Merged
ggwpez
ggwpez approved these changes Jan 24, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@ggwpez ggwpez left a comment

Choose a reason for hiding this comment

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

This also needs the cumulus companion: https://github.com/paritytech/cumulus/pull/2136 in the description.

@ggwpez ggwpez added 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 Feb 10, 2023
@paritytech-ci paritytech-ci requested a review from a team February 11, 2023 06:18
@paritytech-processbot
Copy link
Copy Markdown

paritytech-processbot bot commented Feb 12, 2023

Merge cancelled due to error. Error: Command 'Command { std: "git" "push" "girazoki" "girazoki-companion-set-members-origin", kill_on_drop: false }' failed with status Some(1); output: To https://github.com/girazoki/polkadot.git
! [remote rejected] girazoki-companion-set-members-origin -> girazoki-companion-set-members-origin (refusing to allow a GitHub App to create or update workflow .github/workflows/check-licenses.yml without workflows permission)
error: failed to push some refs to 'https://x-access-token:${SECRET}@github.com/girazoki/polkadot.git'

@bkchr
Copy link
Copy Markdown
Member

bkchr commented Feb 13, 2023

bot merge

@paritytech-processbot
Copy link
Copy Markdown

paritytech-processbot bot commented Feb 13, 2023

Waiting for commit status.

@paritytech-processbot
Copy link
Copy Markdown

paritytech-processbot bot commented Feb 13, 2023

Merge cancelled due to error. Error: Statuses failed for 6b59124

@paritytech-cicd-pr
Copy link
Copy Markdown

paritytech-cicd-pr commented Feb 13, 2023

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/2381518

@bkchr bkchr merged commit 83c5b41 into paritytech:master Feb 13, 2023
tgmichel pushed a commit to moonbeam-foundation/polkadot that referenced this pull request Mar 30, 2023
@girazoki @bkchr @tgmichel
Companion for substrate #13159 (paritytech#6620)
d94d53f 
* SetMembersOrigin

* Update Substrate

* Remove bags list from runtime common

---------

Co-authored-by: Bastian Köcher <[email protected]>
# Conflicts:
#	Cargo.lock
#	runtime/common/Cargo.toml
@kiltbot kiltbot mentioned this pull request Apr 3, 2023
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

4 more reviewers

@gavofyork gavofyork gavofyork approved these changes

@sam0x17 sam0x17 sam0x17 approved these changes

@bkchr bkchr bkchr approved these changes

@ggwpez ggwpez ggwpez approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

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.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@girazoki @bkchr @paritytech-cicd-pr @gavofyork @sam0x17 @ggwpez