refactor: Replace 'meta.ci' -> 'meta.ci.skip'

We need 'meta.ci' to be an attribute set for new CI features.

Change-Id: I83d04e2d74e42e49fe739b049ee4ba799f6d5d86
Reviewed-on: https://cl.tvl.fyi/c/depot/+/5007
Tested-by: BuildkiteCI
Reviewed-by: ezemtsov <eugene.zemtsov@gmail.com>
Reviewed-by: wpcarro <wpcarro@gmail.com>
This commit is contained in:
Vincent Ambo 2022-01-19 20:57:17 +03:00 committed by tazjin
parent 0652ac0ace
commit effcd31651
4 changed files with 5 additions and 5 deletions

View file

@ -65,10 +65,10 @@ let
# To determine build targets, we walk through the depot tree and
# fetch attributes that were imported by readTree and are buildable.
#
# Any build target that contains `meta.ci = false` will be skipped.
# Any build target that contains `meta.ci.skip = true` will be skipped.
# Is this tree node eligible for build inclusion?
eligible = node: (node ? outPath) && (node.meta.ci or true);
eligible = node: (node ? outPath) && (!node.meta.ci.skip or true);
in readTree.fix(self: (readDepot {
depot = self;

View file

@ -47,6 +47,6 @@ self: super: {
# Avoid builds of mkShell derivations in CI.
mkShell = super.lib.makeOverridable(args: (super.mkShell args) // {
meta.ci = false;
meta.ci.skip = true;
});
}

View file

@ -19,5 +19,5 @@ pkgs.stdenv.mkDerivation {
'';
# TODO(wpcarro): This doesn't build at all.
meta.ci = false;
meta.ci.skip = true;
}

View file

@ -20,5 +20,5 @@ pkgs.stdenv.mkDerivation {
'';
# TODO(wpcarro): This doesn't build at all.
meta.ci = false;
meta.ci.skip = true;
}