remove Realized

[MaximilianAlgehed]

@abailly-iohk how do I make the formatter ruin my code? Didn't we used to have a script for that somewhere?

Checklist:

• Check source-code formatting is consistent

[abailly-iohk]

Love removing code! Minor suggestion to deprecate instead of remove Realized

[abailly-iohk]

how do I make the formatter ruin my code? Didn't we used to have a script for that somewhere?

No, we just use fourmolu with the provided fourmolu.yaml file. what's the problem?

[abailly-iohk]

Ah it seems the CI is failing on cabal formatting 🤷

[jorisdral]

Since Realized would be removed, is there a suggested alternative for running tests in a monad other than IO?

[MaximilianAlgehed]

Since Realized would be removed, is there a suggested alternative for running tests in a monad other than IO?

Yes, if you're using IOSim the method is to index the state type by the monad, so if before you had ThreadId in the state type and you were running in IOSim s you'd now have ThreadId m in the state type instead.

[jorisdral]

Since Realized would be removed, is there a suggested alternative for running tests in a monad other than IO?

Yes, if you're using IOSim the method is to index the state type by the monad, so if before you had ThreadId in the state type and you were running in IOSim s you'd now have ThreadId m in the state type instead.

👍 I figured this was probably going to be the suggested method. Thanks!

[MaximilianAlgehed]

I'll remember to put something in the release notes about this!

[MaximilianAlgehed]

@abailly-iohk can we agree - in the light of @jorisdral's comment about -werror - that perhaps deprecating isn't necessary and we should just rip the band-aide off?

[abailly-iohk]

Ah yes, I don't mind although I could argue that an error triggered by -Werror does not qualify as a breaking change and PVP is just wrong, but I won't :) Please revert my change and go ahead removing it

[MaximilianAlgehed]

@abailly-iohk any clue what's up with CI?

[abailly-iohk]

@abailly-iohk any clue what's up with CI?

Some nix f**ery 🤷