pull down to refresh

110 sats \ 2 replies \ @WeAreAllSatoshi 8 Aug 2023 \ on: SN release: nostr login, nostr zap metadata, singular unit(s), more meta
@k00b @ekzyis Is unit testing on y'all's roadmap? I was feeling a little naked making changes without adding some unit tests to go with them. I'd be happy to take a stab at it in a PR if you're interested.
write
preview
31 sats \ 1 reply \ @k00b OP 9 Aug 2023
I'd like to see unit tests around the wallet touching code.
Everything else is kind of nonessential for the time being and the agility is worth the naked feeling. I think the wallet could really benefit from tests though because the stakes are a lot higher yet it'd be nice to iterate there as fast as everywhere else.
reply
10 sats \ 0 replies \ @WeAreAllSatoshi 9 Aug 2023
Makes sense, thanks for the follow up!
reply