Conversation
src/Relude/Extra/Map.hs
Outdated
| @@ -121,19 +121,45 @@ instance StaticMap IntSet where | |||
| {-# INLINE lookup #-} | |||
| {-# INLINE member #-} | |||
There was a problem hiding this comment.
Should we refactor this so that each INLINE would go under the function?
src/Relude/Lifted/IORef.hs
Outdated
|
|
||
| Lifted reexports from 'Data.IORef' module. | ||
|
|
||
| Lifted meaning that you can also use them inside various |
There was a problem hiding this comment.
Let's keep this comment consistent with other identical comments 🙂
| hello | ||
|
|
||
| "hello | ||
| hello" |
There was a problem hiding this comment.
I ended up leaving the " out because in ghci terminal they're not displayed. ¯_(ツ)_/¯
There was a problem hiding this comment.
Maybe this is some strange behaviour of doctest
|
|
||
|
|
||
| -- $setup | ||
| -- >>> import Relude |
There was a problem hiding this comment.
I couldn't get import Relude to work, would always get:
<no location info>: error:
Could not load module ‘Relude’
It is a member of the hidden package ‘relude-0.6.0.0’.
You can run ‘:set -package relude’ to expose it.
(Note: this unloads all the modules in the current scope.)
then follow those instructions and I'd get:
<interactive>:1:1: error:
attempting to use module ‘main:Relude’ (/Users/cmdv/Haskell-OSS/relude/src/Relude.hs) which is not loaded
maybe I have things set up wrong 😄
There was a problem hiding this comment.
Yeah, I can see that it's possible to have invalid config and break doctest...
There was a problem hiding this comment.
yeah not too sure, need to look into it. sorry this was off topic but the PR looks great 🎉
Resolves #173
Now all tests are passing (at least on GHC-8.6.5, let's see what's CI will tell).