Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ci: Auto-determine checks to run #106

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

ci: Auto-determine checks to run #106

wants to merge 3 commits into from

Commits on Jan 14, 2023

  1. checks: do not use fake passthru 

    Exposing passhtru attributes as the derivation attributes is done by `stdenv.mkDerivation` so if we just add them to the passthru attribute of the resulting derivation attrset, they will be only available inside the passthru attribute. We can just append those attributes to the derivation attrset directly, it won’t affect the derivation either way.
    @jtojnar @drupol
    jtojnar authored and drupol committed Jan 14, 2023
    Configuration menu
    Copy the full SHA
    286067b View commit details
    Browse the repository at this point in the history

  2. checks: Move most of the code to flake.nix 

    This will basically allow us to treat `checks.nix` like something that supports having its dependencies injected by `callPackages`, allowing us to simplify the expression a bit. It will also allow other scripts to get the check names trivially.
    @jtojnar @drupol
    jtojnar authored and drupol committed Jan 14, 2023
    Configuration menu
    Copy the full SHA
    9c82c8a View commit details
    Browse the repository at this point in the history

  3. ci: Auto-determine checks to run 

    This will allow us to support multiple platforms easily.
    @jtojnar @drupol
    jtojnar authored and drupol committed Jan 14, 2023
    Configuration menu
    Copy the full SHA
    1b245c7 View commit details
    Browse the repository at this point in the history