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

Create Lurk errors #109

Closed
emmorais opened this issue Jun 13, 2022 · 3 comments
Closed

Create Lurk errors #109

emmorais opened this issue Jun 13, 2022 · 3 comments
Labels
DX-DevEx Developer Experience track invalid This doesn't seem right

Comments

@emmorais
Copy link
Contributor

Solve the following items:

  • Refactor reduce_witness() to return Result<...>
  • Create Lurk errors
  • Replace panics by Lurk errors where applicable
  • Review remaining panics in car_cdr_mut()
  • Refactor car_cdr() as car_cdr_mut()
@emmorais emmorais changed the title Refactor reduce_witness to return Result<...> Create Lurk errors Jun 13, 2022
@MavenRain
Copy link

@emmorais Do you mean reduce_with_witness?

@porcuquine
Copy link
Collaborator

@emmorais Can you review then create issues for any items that are still not done and also don't already have an issue. Once that's done, I think this can be closed.

@huitseeker huitseeker added the DX-DevEx Developer Experience track label May 4, 2023
@huitseeker huitseeker added the invalid This doesn't seem right label Nov 14, 2023
@huitseeker
Copy link
Member

Issues mentioned are either fixed or too vague.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
DX-DevEx Developer Experience track invalid This doesn't seem right
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@huitseeker @porcuquine @MavenRain @emmorais