Skip to content

Commit

Permalink
ci/check: Run checks for all systems (not only x86_64-darwin)
Browse files Browse the repository at this point in the history
  • Loading branch information
GaetanLepage committed Nov 23, 2023
1 parent a88b955 commit 3aaa153
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,6 @@ jobs:
with:
name: nix-community
authToken: '${{ secrets.CACHIX_AUTH_TOKEN }}'
- run: nix flake check
- run: nix flake check ./templates/_wrapper/simple
- run: nix flake check --all-systems --max-jobs 1
- run: nix flake check --all-systems ./templates/_wrapper/simple
- run: nix build .#docs --show-trace
4 changes: 2 additions & 2 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ This document is mainly for contributors to nixvim, but it can also be useful fo
In order to submit a change you must be careful of several points:

- The code must be properly formatted. This can be done through `nix fmt`.
- The tests must pass. This can be done through `nix flake check` (this also checks formatting).
- The tests must pass. This can be done through `nix flake check --all-systems` (this also checks formatting).
- The change should try to avoid breaking existing configurations.
- If the change introduces a new feature it should add tests for it (see the architecture section for details).

Expand Down Expand Up @@ -104,7 +104,7 @@ You can specify the special `tests` attribute in the configuration that will not

- `tests.dontRun`: avoid launching this test, simply build the configuration.

The tests are then runnable with `nix flake check`.
The tests are then runnable with `nix flake check --all-systems`.

There are a second set of tests, unit tests for nixvim itself, defined in `tests/lib-tests.nix` that use the `pkgs.lib.runTests` framework.

Expand Down

0 comments on commit 3aaa153

Please sign in to comment.