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

[ci] remove cargo-check-nixos job #11873

Merged
merged 2 commits into from
Jul 26, 2022
Merged

[ci] remove cargo-check-nixos job #11873

merged 2 commits into from
Jul 26, 2022

Conversation

alvicsam
Copy link
Contributor

The PR removes cargo-check-nixos also add comments for check-tracing and test-full-crypto-feature jobs.

More information can be found in https://github.com/paritytech/ci_cd/issues/506

@alvicsam alvicsam requested a review from a team as a code owner July 20, 2022 16:53
@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Jul 20, 2022
@alvicsam alvicsam added 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 Jul 20, 2022
@ggwpez
Copy link
Member

ggwpez commented Jul 20, 2022

If we remove the cargo-check-nixos, then we should rather also remove the shell.nix since it WILL go out of sync again and people will open issues about it.
cc @bkchr

@paritytech-ci paritytech-ci requested a review from a team July 26, 2022 08:10
@alvicsam alvicsam merged commit 531d7c3 into master Jul 26, 2022
@alvicsam alvicsam deleted the as-ci-chores branch July 26, 2022 09:37
DaviRain-Su pushed a commit to octopus-network/substrate that referenced this pull request Aug 23, 2022
* [ci] remove cargo-check-nixos job

* remove shell.nix
ark0f pushed a commit to gear-tech/substrate that referenced this pull request Feb 27, 2023
* [ci] remove cargo-check-nixos job

* remove shell.nix
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.

5 participants