-
Notifications
You must be signed in to change notification settings - Fork 33
Conversation
e5883b8
to
0cbaaa8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really don't like that traverse eval
, but omg define
s work!
src/Radicle/Internal/Core.hs
Outdated
modEnv = mappings <> closure | ||
NonEmpty.last <$> withEnv (const modEnv) | ||
(traverse eval body) | ||
res <- NonEmpty.last <$> withEnv (const modEnv) (traverse eval body) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should it not be traverse baseEval
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And furthermore, can it reuse callFn
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
#t | ||
|] | ||
prog `succeedsWith` Boolean False | ||
|
||
, testCase "redefining eval keeps access to future definitions" $ do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯 🥇
72ea52e
to
6b3033d
Compare
See #98 .