You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
causes code that uses it to be removed when compiling using the backend-optimizer. This is problematic when implementing a new backend where the backend-optimizer is of great benefit.
I suggest we replace the use of AlmostEff with just enough Effect to prevent this from happening.
Even if not using the backend-optimizer or somehow making it possible to prevent dead code elimination I think it would still be better not to use Unit -> Unit as the type in the tests.
The text was updated successfully, but these errors were encountered:
The type
causes code that uses it to be removed when compiling using the backend-optimizer. This is problematic when implementing a new backend where the backend-optimizer is of great benefit.
I suggest we replace the use of
AlmostEff
with just enoughEffect
to prevent this from happening.Even if not using the backend-optimizer or somehow making it possible to prevent dead code elimination I think it would still be better not to use
Unit -> Unit
as the type in the tests.The text was updated successfully, but these errors were encountered: