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

Fix CI benchmark check #8380

Merged
merged 1 commit into from
Mar 16, 2021
Merged

Fix CI benchmark check #8380

merged 1 commit into from
Mar 16, 2021

Conversation

bkchr
Copy link
Member

@bkchr bkchr commented Mar 16, 2021

No description provided.

@bkchr bkchr 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 Mar 16, 2021
@bkchr bkchr requested a review from TriplEight March 16, 2021 17:57
@bkchr bkchr requested a review from athei as a code owner March 16, 2021 17:57
@bkchr bkchr added the D1-audited 👍 PR contains changes to fund-managing logic that has been properly reviewed and externally audited label Mar 16, 2021
@bkchr bkchr merged commit 66098c2 into master Mar 16, 2021
@bkchr bkchr deleted the bkchr-fix-benchmark-check branch March 16, 2021 18:37
TriplEight added a commit that referenced this pull request Mar 16, 2021
* CI: run cargo deny only nightly before it's fixed

* Release missing 3.0 crates: pallet-node-authorization sc-finality-grandpa-warp-sync (#8360)

* bump pallet-node-authorization

* prepping sc-finality-grandpa-warp-sync for release

* bump Cargo.lock

* Update ss58-registry.json (#8351)

Added Plasm Network description

* [CI] Require D*-audit labels for any runtime changes (#8345)

* add check for audit labels if runtime change

* fix shellcheck nits

* include lib.sh in check_runtime.sh

* fix check_labels.sh

* fix check_labels.sh

* oops, this is github actions...

* why wont this work

* fetch all refs

* Update check-labels.yml

* print env - wtf is happening

* checkout the PR...

* ffs

* fix

* REVERT ME: test runtime check

* Revert "REVERT ME: test runtime check"

This reverts commit 0fd2b04.

* Fix CI benchmark check (#8380)

* CI: run cargo deny only nightly before it's fixed

Co-authored-by: Benjamin Kampmann <[email protected]>
Co-authored-by: Aleksandr Krupenkin <[email protected]>
Co-authored-by: Martin Pugh <[email protected]>
Co-authored-by: Bastian Köcher <[email protected]>
hirschenberger pushed a commit to hirschenberger/substrate that referenced this pull request Apr 14, 2021
hirschenberger pushed a commit to hirschenberger/substrate that referenced this pull request Apr 14, 2021
* CI: run cargo deny only nightly before it's fixed

* Release missing 3.0 crates: pallet-node-authorization sc-finality-grandpa-warp-sync (paritytech#8360)

* bump pallet-node-authorization

* prepping sc-finality-grandpa-warp-sync for release

* bump Cargo.lock

* Update ss58-registry.json (paritytech#8351)

Added Plasm Network description

* [CI] Require D*-audit labels for any runtime changes (paritytech#8345)

* add check for audit labels if runtime change

* fix shellcheck nits

* include lib.sh in check_runtime.sh

* fix check_labels.sh

* fix check_labels.sh

* oops, this is github actions...

* why wont this work

* fetch all refs

* Update check-labels.yml

* print env - wtf is happening

* checkout the PR...

* ffs

* fix

* REVERT ME: test runtime check

* Revert "REVERT ME: test runtime check"

This reverts commit 0fd2b04.

* Fix CI benchmark check (paritytech#8380)

* CI: run cargo deny only nightly before it's fixed

Co-authored-by: Benjamin Kampmann <[email protected]>
Co-authored-by: Aleksandr Krupenkin <[email protected]>
Co-authored-by: Martin Pugh <[email protected]>
Co-authored-by: Bastian Köcher <[email protected]>
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. D1-audited 👍 PR contains changes to fund-managing logic that has been properly reviewed and externally audited
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants