Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

remove all the TODOs #52

Open
Tracked by #778
Fi3 opened this issue Nov 16, 2021 · 1 comment
Open
Tracked by #778

remove all the TODOs #52

Fi3 opened this issue Nov 16, 2021 · 1 comment
Assignees
@Fi3 @ccdle12 @rrybarczyk
Labels
test Test related
Milestone
Error(s) refactor...

Comments

@Fi3
Copy link
Collaborator

Fi3 commented Nov 16, 2021

No description provided.

@Fi3 Fi3 mentioned this issue Nov 16, 2021
Closed
3 tasks
@Fi3 Fi3 changed the title remove all the TODOs that means that the TODOs are either:1. resolved2. removed3. transformed into github issues remove all the TODOs Nov 16, 2021
@rrybarczyk rrybarczyk added the test Test related label Sep 16, 2022
@rrybarczyk
Copy link
Collaborator

We have been removing a lot of the Error variant Todos, still may be some.

Remove/handle all:

map_err(|_| Err(())

@plebhash plebhash mentioned this issue May 4, 2024
Open
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Fi3 Fi3

@ccdle12 ccdle12

@rrybarczyk rrybarczyk

Labels
test Test related
Projects
No open projects
Sv2 Project Plan 2022
Status: In Progress 🏗️
Milestone
Error(s) refactored, good code covera...
Development

No branches or pull requests
3 participants
@Fi3 @ccdle12 @rrybarczyk