IntersectMBO / IntersectMBO/ouroboros-consensus

Check the `NoThunks` invariant during a nightly CI test

Open
#604 4 comments 0 reactions 1 assignee Claimed by @fraser-iohk View on GitHub
technical debt
Dominant language
Haskell
Stars
67
Forks
43
Avg merge
5d 13h
Merged PRs (30d)
43

Description

If you set the following flag and run the test suite, most `TVar`s -- including all critical ones, unless we have a bug -- will check their contents for thunks via the `nothunks` library whenever they're updated.

```
$ cat cabal.project.local
package io-classes
flags: +checktvarinvariant
```

This flag makes the tests run quite slow, so it's something we had been doing incrementally. But I, for example, haven't done it since taking over lead for two months! So adding it to some CI (nightly, but perhaps even just weekly) would be much safer.

Edit: `io-sim-classes` was renamed `io-classes` on 2021 July 8.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.