Skip to content

Conversation

@mordamax
Copy link
Contributor

@mordamax mordamax commented Aug 5, 2022

Removing cargo check because it will be a part of resulted PR on substrate-*-template.
Few reasons:

  • If the cargo check fails here - it has no visibility to owners, if on PR - owners will analyze the problem from there
  • it's taking too long from this CI and in success case - it will be run twice here and on PR level, which is redundant

@mordamax mordamax requested a review from a team as a code owner August 5, 2022 13:53
@mordamax mordamax merged commit b8ccce6 into master Aug 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants