serokell / coffer

Multi-backend password store with multiple frontends
4 stars 2 forks source link

[#26] Fix `validate cabal files` CI step #77

Closed dcastro closed 2 years ago

dcastro commented 2 years ago

Description

Problem: The validate cabal files build step hasn't been working ever since we added it. We can see this message in the logs:

./scripts/validate-cabal-files.sh: line 39: diff: command not found

The reason why the build step did not actually ever fail (thus, it was a false negative) is because we temporarily use set +e to mask all non-zero exit codes.

Solution: add the missing diffTools to the nix shell.

Related issue(s)

Fixed part of #26

:white_check_mark: Checklist for your Pull Request

Related changes (conditional)

Stylistic guide (mandatory)