Thomas Winant
Thomas Winant
It seems this project has no CI. Before submitting this PR, I successfully compiled this branch with all `stack-*.yaml` files in this repo (hence the choice for [`base64-bytestring`](https://hackage.haskell.org/package/base64-bytestring) library over...
I'm all for it!
Some suggestions for names: * `ouroboros-common`: for the modules that both network and consensus need, e.g., `Ouroboros.Network.Block`, `Ouroboros.Network.ChainFragment`, etc. We can use the `Ouroboros.*` or `Ouroboros.Common.*` module hierarchy. * `ouroboros-node`:...
@edsko pointed out that it might make sense to change the `ouroboros-` prefix to `cardano-` (e.g., `cardano-network` and `cardano-consensus`) which is not a bad idea. We should then also change...
> There's nothing specific to `cardano` in `ouroboros-network` but there's also nothing specific to `ouroboros` either. We could use some fancy name instead. Yeah, Edsko and I realised too that...
Thought: we could rename `ouroboros-consensus-cardano` to `cardano-consensus`. I think it would also be good for consensus to have some API-like modules that export all things downstream users might need. For...
_TL;DR: try updating to a newer version of Docker._ @bsaul I ran into the same issue! Glad that I'm not the only one thinking they're going mad :slightly_smiling_face: Our CI,...
Copying the list from #670: - [ ] `ChainValidationInvalidSignature` - [ ] `ChainValidationBlockAttributesTooLarge` - [ ] `ChainValidationHeaderAttributesTooLarge` - [ ] `ChainValidationDelegationPayloadError` - [ ] `ChainValidationInvalidDelegation` - [ ] `ChainValidationMissingHash` -...
Should the `TxWitness` also be included in the hash?
> > I believe so, yes. > > I guess I'm wrong about this. > > ``` > -- | Represents transaction identifier as 'Hash' of 'Tx' > type TxId...