Conversation
Most parachains-related code is within a separate folder |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, with comments.
scripts/gitlab/check_runtime.sh
Outdated
@@ -13,77 +13,75 @@ | |||
# runtime/{polkadot,kusama}/src/lib.rs file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
needs updating
|
||
# Pretty-printing functions | ||
boldprint () { printf "|\n| \033[1m%s\033[0m\n|\n" "${@}"; } | ||
boldcat () { printf "|\n"; while read -r l; do printf "| \033[1m%s\033[0m\n" "${l}"; done; printf "|\n" ; } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice finally made it into the lib :)
Hmm good point, will add that in, cheers |
@rphmeier requested changes implemented :) As soon as you give this a thumbs up I'll merge. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LvGTM
Thanks! |
This PR aims to redo check_runtime.sh to better support multiple runtimes. It defines a list of runtimes (currently kusama, polkadot and westend).
Outline of the new logic:
runtime/$runtime/*
) orruntime/common/*
, then look for any changes to Substrate between the last tag and the current commit. If any of them are labelled D1-breaksapi, then give this PR the same label. Then exit.spec_version
orimpl_version
have not been bumped for that specific runtime since the last release, the test will fail and block merging the PR.master
, label the PR with D2-breaksapiref https://github.com/paritytech/devops/issues/557