$ /nix/store/9ypz3flqsrl5xl495mm8h645gadjsxi1-coreutils-9.11/bin/timeout --kill-after=15s 7200s /nix/store/23zk8sgzamrbmj1fday15szf7f2279s8-nix-2.34.7/bin/nix --extra-experimental-features nix-command --extra-experimental-features flakes --log-format internal-json build --no-link --print-out-paths git+https://x-access-token:@github.com/NorfairKing/hopinion?ref=initial&rev=499671eb3ca5305986dc0c38561eb9dd8b1753d2&shallow=1#checks.x86_64-linux.e2e-dirty-project-is-judged-failing --print-build-logs  Building /nix/store/yf35rdc0wzhk9j7lvlnp541pn3ms3z06-lonely-hopinion.drv Building /nix/store/mqbz5w1v6qlhgwb83a6hf37z574d7xbb-thing-gen-hopinion.drv [lonely-hopinion] [error HsGenValidInGenPackage]: A GenValid instance outside a -gen package. [lonely-hopinion]  ╭──▶ lonely/src/Lonely.hs@8:1-8:25 [lonely-hopinion]  │ [lonely-hopinion]  8 │ instance GenValid Orphan [lonely-hopinion]  • ┬─────────────────────── [lonely-hopinion]  • ╰╸ Put generators (like GenValid instances) in the -gen package [lonely-hopinion]  • [lonely-hopinion]  │ Note: From haskell-style.md. A GenValid instance in a package's own library gets compiled into the executable. Move it to the matching -gen package. [lonely-hopinion]  │ Note: A generator is test code, and an instance in the library is linked into everything that depends on the library, so the shipped executable carries QuickCheck and the generators with it. The -gen package exists so that dependency stops at the test suite, which is also what keeps a generator free to be as slow or as elaborate as the property needs. [lonely-hopinion]  │ Hint: To suppress, write this directly above line 8 of lonely/src/Lonely.hs, and say why: -- [allow:HsGenValidInGenPackage]  [lonely-hopinion] ─────╯ [lonely-hopinion:post-build] Uploading to the NixCI staging cache: /nix/store/9ig8g31cxgcmarfddbvdfk524dij350f-lonely-hopinion [lonely-hopinion:post-build] warning: 'warn-short-path-literals' is deprecated, use 'lint-short-path-literals = ignore' instead [lonely-hopinion:post-build] copying 1 paths... [lonely-hopinion:post-build] copying path '/nix/store/9ig8g31cxgcmarfddbvdfk524dij350f-lonely-hopinion' to 'https://cache.staging.nix-ci.com'... [lonely-hopinion:post-build] warning: 'warn-short-path-literals' is deprecated, use 'lint-short-path-literals = ignore' instead [lonely-hopinion:post-build] copying 0 paths... Uploaded lonely-hopinion in 418ms Progress: 1 of 5 built (1 building) Built lonely-hopinion in 533ms [thing-gen-hopinion:post-build] Uploading to the NixCI staging cache: /nix/store/gzd5lckyzi9y1iajzjjw2m61i684f796-thing-gen-hopinion [thing-gen-hopinion:post-build] warning: 'warn-short-path-literals' is deprecated, use 'lint-short-path-literals = ignore' instead [thing-gen-hopinion:post-build] copying 1 paths... [thing-gen-hopinion:post-build] copying path '/nix/store/gzd5lckyzi9y1iajzjjw2m61i684f796-thing-gen-hopinion' to 'https://cache.staging.nix-ci.com'... [thing-gen-hopinion:post-build] warning: 'warn-short-path-literals' is deprecated, use 'lint-short-path-literals = ignore' instead [thing-gen-hopinion:post-build] copying 0 paths... Uploaded thing-gen-hopinion in 400ms Progress: 2 of 5 built Built thing-gen-hopinion in 918ms Building /nix/store/0lnkd7qcw3clqvirjznam52hihyjymcw-hopinion-project.drv [hopinion-project] [error HsGenValidInGenPackage]: A GenValid instance outside a -gen package. [hopinion-project]  ╭──▶ lonely/src/Lonely.hs@8:1-8:25 [hopinion-project]  │ [hopinion-project]  8 │ instance GenValid Orphan [hopinion-project]  • ┬─────────────────────── [hopinion-project]  • ╰╸ Put generators (like GenValid instances) in the -gen package [hopinion-project]  • [hopinion-project]  │ Note: From haskell-style.md. A GenValid instance in a package's own library gets compiled into the executable. Move it to the matching -gen package. [hopinion-project]  │ Note: A generator is test code, and an instance in the library is linked into everything that depends on the library, so the shipped executable carries QuickCheck and the generators with it. The -gen package exists so that dependency stops at the test suite, which is also what keeps a generator free to be as slow or as elaborate as the property needs. [hopinion-project]  │ Hint: To suppress, write this directly above line 8 of lonely/src/Lonely.hs, and say why: -- [allow:HsGenValidInGenPackage]  [hopinion-project] ─────╯ [hopinion-project]  [hopinion-project] [error TestGenValidSpecPerGenValid]: No genValidSpec @Orphan anywhere, and there is no lonely-gen for it to be written in. [hopinion-project]  ╭──▶ lonely/src/Lonely.hs@8:1-8:25 [hopinion-project]  │ [hopinion-project]  8 │ instance GenValid Orphan [hopinion-project]  • ┬─────────────────────── [hopinion-project]  • ╰╸ For every GenValid instance, have a genValidSpec @FooBar [hopinion-project]  • [hopinion-project]  │ Note: From testing.md. A GenValid instance creates an obligation: genValidSpec @T must be called in the test suite of that package's gen package. [hopinion-project]  │ Note: A generator nothing runs is a generator nobody has checked. genValid can produce values the type's own Validity instance rejects, and shrinkValid can shrink to them, and every property built on it inherits that quietly: the failures it reports are about values the code was never meant to see. genValidSpec asserts exactly the two things the instance promises, and it is one line. The spec belongs to the package that declares the instance, because a spec written next door disappears the day that other package stops mentioning the type. [hopinion-project]  │ Hint: To suppress, write this directly above line 8 of lonely/src/Lonely.hs, and say why: -- [allow:TestGenValidSpecPerGenValid]  [hopinion-project] ─────╯ [hopinion-project]  [hopinion-project] [error CommentBareTodo]: A bare TODO. Reference an issue or a URL, or do it. [hopinion-project]  ╭──▶ thing/src/Thing.hs@3:1-3:58 [hopinion-project]  │ [hopinion-project]  3 │ -- TODO a bare marker, which is the module-level finding. [hopinion-project]  • ┬──────────────────────────────────────────────────────── [hopinion-project]  • ╰╸ A bare TODO: do it, reference an issue, or say concretely what is missing [hopinion-project]  • [hopinion-project]  │ Note: From comments.md. A TODO needs an issue reference or a URL saying where the follow-up is tracked. [hopinion-project]  │ Note: A TODO nothing points at is a note nobody is holding. It reads like a plan, so the next reader assumes it is one, and it goes stale in place because nothing brings it back up. A reference makes it findable from wherever the work is actually tracked; doing it makes the note unnecessary. If it is neither tracked nor worth doing, the honest version is a comment that says what is missing and what would make it worth doing. [hopinion-project]  │ Hint: To suppress, write this directly above line 4 of thing/src/Thing.hs, and say why: -- [allow:CommentBareTodo]  [hopinion-project] ─────╯ [hopinion-project]  [hopinion-project] [error HsGenValidInGenPackage]: A GenValid instance outside a -gen package. [hopinion-project]  ╭──▶ thing/src/Thing.hs@9:1-9:25 [hopinion-project]  │ [hopinion-project]  9 │ instance GenValid Widget [hopinion-project]  • ┬─────────────────────── [hopinion-project]  • ╰╸ Put generators (like GenValid instances) in the -gen package [hopinion-project]  • [hopinion-project]  │ Note: From haskell-style.md. A GenValid instance in a package's own library gets compiled into the executable. Move it to the matching -gen package. [hopinion-project]  │ Note: A generator is test code, and an instance in the library is linked into everything that depends on the library, so the shipped executable carries QuickCheck and the generators with it. The -gen package exists so that dependency stops at the test suite, which is also what keeps a generator free to be as slow or as elaborate as the property needs. [hopinion-project]  │ Hint: To suppress, write this directly above line 9 of thing/src/Thing.hs, and say why: -- [allow:HsGenValidInGenPackage]  [hopinion-project] ─────╯ [hopinion-project]  [hopinion-project] [error TestGenValidSpecPerGenValid]: No genValidSpec @Widget anywhere in thing-gen's test suite. [hopinion-project]  ╭──▶ thing/src/Thing.hs@9:1-9:25 [hopinion-project]  │ [hopinion-project]  9 │ instance GenValid Widget [hopinion-project]  • ┬─────────────────────── [hopinion-project]  • ╰╸ For every GenValid instance, have a genValidSpec @FooBar [hopinion-project]  • [hopinion-project]  │ Note: From testing.md. A GenValid instance creates an obligation: genValidSpec @T must be called in the test suite of that package's gen package. [hopinion-project]  │ Note: A generator nothing runs is a generator nobody has checked. genValid can produce values the type's own Validity instance rejects, and shrinkValid can shrink to them, and every property built on it inherits that quietly: the failures it reports are about values the code was never meant to see. genValidSpec asserts exactly the two things the instance promises, and it is one line. The spec belongs to the package that declares the instance, because a spec written next door disappears the day that other package stops mentioning the type. [hopinion-project]  │ Hint: To suppress, write this directly above line 9 of thing/src/Thing.hs, and say why: -- [allow:TestGenValidSpecPerGenValid]  [hopinion-project] ─────╯ [hopinion-project:post-build] Uploading to the NixCI staging cache: /nix/store/inaqlm4yi7y78ai4kx8v36lfiz9y1pcb-hopinion-project [hopinion-project:post-build] warning: 'warn-short-path-literals' is deprecated, use 'lint-short-path-literals = ignore' instead [hopinion-project:post-build] copying 1 paths... [hopinion-project:post-build] copying path '/nix/store/inaqlm4yi7y78ai4kx8v36lfiz9y1pcb-hopinion-project' to 'https://cache.staging.nix-ci.com'... [hopinion-project:post-build] warning: 'warn-short-path-literals' is deprecated, use 'lint-short-path-literals = ignore' instead [hopinion-project:post-build] copying 0 paths... Uploaded hopinion-project in 411ms Progress: 3 of 5 built Built hopinion-project in 643ms Building /nix/store/59nmifmaq95p0afi0lixqri0j3zylsg5-e2e-dirty-project-is-judged-failing.drv [e2e-dirty-project-is-judged-failing:post-build] Uploading to the NixCI staging cache: /nix/store/hjs80vj2bq4p2vl2qczl5x9lbgi6d1qw-e2e-dirty-project-is-judged-failing [e2e-dirty-project-is-judged-failing:post-build] warning: 'warn-short-path-literals' is deprecated, use 'lint-short-path-literals = ignore' instead [e2e-dirty-project-is-judged-failing:post-build] copying 1 paths... [e2e-dirty-project-is-judged-failing:post-build] copying path '/nix/store/hjs80vj2bq4p2vl2qczl5x9lbgi6d1qw-e2e-dirty-project-is-judged-failing' to 'https://cache.staging.nix-ci.com'... [e2e-dirty-project-is-judged-failing:post-build] warning: 'warn-short-path-literals' is deprecated, use 'lint-short-path-literals = ignore' instead [e2e-dirty-project-is-judged-failing:post-build] copying 0 paths... Uploaded e2e-dirty-project-is-judged-failing in 443ms Progress: 4 of 5 built Built e2e-dirty-project-is-judged-failing in 513ms Progress: 5 of 5 built /nix/store/hjs80vj2bq4p2vl2qczl5x9lbgi6d1qw-e2e-dirty-project-is-judged-failing Build succeeded.