Skip to content

Commit

Permalink
chore: disable nix tests
Browse files Browse the repository at this point in the history
For a few reasons

1. This step is really, really slow. I don't think there's sufficient
   value in a slow CI step to keep it
2. Whenever we add an integration test it needs to be added to the
   ignore list. I want to keep friction on adding such tests as low as
   is possible.
3. We already run tests in a bunch of places, so I don't think this is
   needed

Ref: #1123
  • Loading branch information
ellie committed Jan 29, 2024
1 parent 0faf414 commit 1229aac
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions atuin.nix
Original file line number Diff line number Diff line change
Expand Up @@ -36,12 +36,7 @@ rustPlatform.buildRustPackage {
--zsh <($out/bin/atuin gen-completions -s zsh)
'';

# Additional flags passed to the cargo test binary, see `cargo test -- --help`
checkFlags = [
# Sync tests require a postgres server
"--skip=sync"
"--skip=registration"
];
doCheck = false;

meta = with lib; {
description = "Replacement for a shell history which records additional commands context with optional encrypted synchronization between machines";
Expand Down

0 comments on commit 1229aac

Please sign in to comment.