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

Conversation

@gilescope
Copy link
Contributor

We need to slurp collectives if we want the chain to appear in the release notes.

@gilescope gilescope added A3-inprogress B0-silent Changes should not be mentioned in any release notes labels Jul 27, 2022
@gilescope gilescope requested review from a team and chevdor as code owners July 27, 2022 09:59
@gilescope
Copy link
Contributor Author

Hmm, it's not too happy

D, [2022-07-27T10:49:07.616018 #1741] DEBUG -- : Building changelog with runtimes
jq: error: $srtool_polkadot_collectives_parachain is not defined at <top-level>, line 6:
          { name: "polkadot-collectives", data: $srtool_polkadot_collectives_parachain[0] },                                                
jq: 1 compile error
thread 'main' panicked at 'JSON parsing: Error("EOF while parsing a value", line: 1, column: 0)', src/wrapped_context.rs:40:53
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Aborted (core dumped)

@chevdor
Copy link
Contributor

chevdor commented Jul 27, 2022

srtool_polkadot_collectives_parachain vs srtool_polkadot_collectives

@chevdor chevdor requested a review from a team August 4, 2022 10:24
@joepetrowski joepetrowski merged commit 8d104b6 into joe-alliance Aug 4, 2022
@joepetrowski joepetrowski deleted the giles-alliance-add-slurp branch August 4, 2022 15:48
paritytech-processbot bot pushed a commit that referenced this pull request Aug 11, 2022
* add new runtime and remove unnecessary pallets

* make runtime build

* add collectives to collator node

* sketch alliance config in runtime

* Slash handler was supposed to be commented out (for now)

* correct signature

* move to impls

* add alliance to runtime

* rustfmt

* IsReserve, remove Ping, update fn deposit

* add transaction_payment event

* Update parachains/runtimes/collectives/collectives-polkadot/src/lib.rs

Co-authored-by: Squirrel <[email protected]>

* fmt

* add genesis config to chain spec

* fix merge

* local and dev configs only (for now)

* remove duplicate imports

* Collectives polkadot runtime to cargo workspace members (#1397)

* Collectives polkadot runtime: use unit type impl for identity verifier (#1398)

* apply fn rename

* fmt

* one less todo

* Less code in magic macros (#1407)

* Less code in magic macros

* cargo fmt

* Bench alliance (#1427)

* add benchmarks

* call one script from the other

* shebang changes so works on nixos too.

* bench in parallel as separate jobs

* hyphens can turn into underscores

* remove workaround to trigger bench

Co-authored-by: alvicsam <[email protected]>
Co-authored-by: paritytech-ci <[email protected]>

* enable ci jobs

* fix publish bench results jobs

* chainspecs for collectives-westend (#1441)

* initial chainspecs for collections relay chain

* plumb in the collectives-westend chainspec

* add Runtime::CollectivesWestend

* lock

* Collectives: teleport slashed assets  (#1433)

* Collectives: teleport slashed assets

* fmt

* Cargo.lock > polkadot-parachain 0.9.25

* create temp account for imbalance

* treasury acc id from pallet id

* move accounts into constants, use here junction for assets

* assets location is relay chain, accounts as parameters

* fix typos

* fix typo

* Update parachains/runtimes/collectives/collectives-polkadot/src/constants.rs

Co-authored-by: joe petrowski <[email protected]>

* Move alliance proposal provider to impls.rs (#1464)

* Move to impls alliance proposal provider

* rustfmt

* Bumping spec version

(so that we can redeploy with slashing change.)

* cargo lock

* slurp collectives digest to make appear in release notes (#1473)

* add slurp

* Slurp better :)

* Bring some order

Co-authored-by: Chevdor <[email protected]>
Co-authored-by: Wilfried Kopp <[email protected]>

* reorder barrier

* Update parachains/runtimes/collectives/collectives-polkadot/src/xcm_config.rs

Co-authored-by: Keith Yeung <[email protected]>

* address review

* clean construct runtime

* fmt

* looks pretty but brings in too much

Co-authored-by: Squirrel <[email protected]>
Co-authored-by: Muharem Ismailov <[email protected]>
Co-authored-by: alvicsam <[email protected]>
Co-authored-by: paritytech-ci <[email protected]>
Co-authored-by: Chevdor <[email protected]>
Co-authored-by: Wilfried Kopp <[email protected]>
Co-authored-by: Keith Yeung <[email protected]>
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants