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

CI: test and update ci image, codeowners #8142

Merged
merged 13 commits into from
Feb 19, 2021
Merged

Conversation

TriplEight
Copy link
Contributor

@TriplEight TriplEight commented Feb 17, 2021

  • subscribe CI team to the changes CI-related files
  • prepare to nightly trigger this CI with testing substrate CI image CI: trigger substrate CI on image test scripts#258
  • remove diener installation. It's installed in CI image.
  • trigger build and publish jobs on schedules
  • trigger cargo-deny on changes in manifests and lock
  • remove flaming-fir deployment, it will be automated based on updates in s3 buckets
  • trigger simnet with the exact substrate version. cc @radupopa2010
  • remove cargo-audit as it's eclipsed by cargo-deny

@TriplEight TriplEight self-assigned this Feb 17, 2021
@TriplEight TriplEight 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. labels Feb 17, 2021
@TriplEight TriplEight changed the title 3x8_update_ci_images WIP: CI: test and update ci image, codeowners Feb 17, 2021
@TriplEight TriplEight changed the title WIP: CI: test and update ci image, codeowners CI: test and update ci image, codeowners Feb 18, 2021
Copy link
Contributor

@gabreal gabreal left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@s3krit s3krit left a comment

Choose a reason for hiding this comment

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

LGTM 👍 some cool changes in here

@TriplEight TriplEight merged commit 9029ab0 into master Feb 19, 2021
@TriplEight TriplEight deleted the 3x8_update_ci_images branch February 19, 2021 14:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants