record Nick's comments on hs ledger bindings #2340
Conversation
|
|
||
| To use the bindings in this repo, you need the following `BUILD` dep: | ||
| ``` | ||
| "//language-support/hs/bindings:hs-ledger", |
There was a problem hiding this comment.
I would make the in this repo part bold and maybe point out explicitly that this is only for this repo since I don’t expect that most people will use it in the repo.
There was a problem hiding this comment.
Good point. I've also switched the order, which I believe reads better as a logical sequence: how the code is structured, how to use in repo, how to use out of repo. (As opposed to previous commit which had how to use outside, how code is structured, how to use inside.)
garyverhaegen-da
force-pushed
the
hs-bindings-readme
branch
from
2e8f86c to
b652cd4
Compare
|
|
||
| You can find some usage examples | ||
| [here](/language-support/hs/bindings/test/DA/Ledger/Tests.hs) and | ||
| [here](/language-support/hs/bindings/examples/chat/src/DA/Chat/ChatLedger.hs). |
There was a problem hiding this comment.
This link is broken for me
There was a problem hiding this comment.
It should be /language-support/hs/bindings/examples/chat/src/DA/Ledger/App/Chat/ChatLedger.hs
There was a problem hiding this comment.
Thanks for catching this!
There was a problem hiding this comment.
Fixed in #2349.
|
|
||
| You can find some usage examples | ||
| [here](/language-support/hs/bindings/test/DA/Ledger/Tests.hs) and | ||
| [here](/language-support/hs/bindings/examples/chat/src/DA/Chat/ChatLedger.hs). |
There was a problem hiding this comment.
It should be /language-support/hs/bindings/examples/chat/src/DA/Ledger/App/Chat/ChatLedger.hs
@nickchapman-da's response to @bame-da asking about Haskell ledger bindings documentation on Slack, with small adjustments for GitHub formatting.