Skip to content
This repository has been archived by the owner on Aug 18, 2020. It is now read-only.

[CBR-227] Go over TODOs, remove stray, add tickets #3393

Merged
merged 1 commit into from
Aug 14, 2018

Conversation

edsko
Copy link
Contributor

@edsko edsko commented Aug 14, 2018

This only changes comments (and a few variable names). I went over the TODOs still in the code, removed ones that were no longer relevant or had already been done, and created tickets for the important ones that still remain. TODOs about coding style or that sort of thing I just left.

Copy link
Contributor

@uroboros uroboros left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ready to go

@edsko edsko merged commit 39a9019 into develop Aug 14, 2018
@edsko edsko deleted the check-all-wallet-TODOs branch August 14, 2018 13:28
@edsko
Copy link
Contributor Author

edsko commented Aug 14, 2018

Thanks everyone :)

KtorZ pushed a commit that referenced this pull request Nov 9, 2018
[CBR-227] Go over TODOs, remove stray, add tickets
KtorZ pushed a commit to input-output-hk/cardano-wallet-legacy that referenced this pull request Nov 9, 2018
…hk/check-all-wallet-TODOs

[CBR-227] Go over TODOs, remove stray, add tickets
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants