Daniel Kronovet
Daniel Kronovet
@arrenv that should be fine, curious what @area thinks
@area I hear that, but I'm also noting that the addition of rejection functionality adds many hundreds of lines to the contract+tests, whereas a timeout-based rejection would probably require a...
@area do you mean the test [here](https://github.com/JoinColony/colonyNetwork/pull/1144/files#diff-21b9ca4704e7ec0ae6d3a6b7979bcbcb742a74c85e88307d3c9a651660294d55R346)? At first glance I agree with you... it seems strange.
FWIW codebase migrations to TS is the sort of thing I'd expect LLMs to be able to start handling soon. https://about.grit.io/
Hardhat resources: https://hardhat.org/hardhat-runner/docs/other-guides/truffle-migration
Closed with #1194
Unsure what this issue is referring to, as `StreamingPayments.sol` already has three events: - `event StreamingPaymentCreated` - `event StreamingPaymentClaimed` - `event PaymentTokenUpdated`
Noting that the latest version of hardhat (2.21) introduced a new dependency:[ @nomicfoundation/edr](https://www.npmjs.com/package/@nomicfoundation/edr) which seems to break our build. So pinning to version 2.19 with a tilde for now.
~Regarding the version check script, I'm not 100% sure. Given that the "last release" will run on truffle and the "current" version will run on hardhat we may need to...
@area it's pretty spotty in CI for me, re-running the build usually sorts it. I've been ignoring it for the time being.