Skip to content
This repository was archived by the owner on Dec 8, 2024. It is now read-only.

Review Drake modules #16

Open
3 tasks done
OnofreTZK opened this issue May 20, 2023 · 1 comment
Open
3 tasks done

Review Drake modules #16

OnofreTZK opened this issue May 20, 2023 · 1 comment
Assignees
@OnofreTZK
Labels
Sheldrake: improvement improve the code or infra Sheldrake: invalid business logic This doesn't seem right Sheldrake: question Further information is requested

Comments

@OnofreTZK
Copy link
Collaborator

OnofreTZK commented May 20, 2023 •
edited
Loading

Tasks

  • Dispose Mempool and Chain\
  • Rewrite Transaction and Block to adapt to t list
  • Genesis block at first migration
@OnofreTZK OnofreTZK added Sheldrake: improvement improve the code or infra Sheldrake: invalid business logic This doesn't seem right Sheldrake: question Further information is requested labels May 20, 2023
@OnofreTZK OnofreTZK self-assigned this May 20, 2023
@OnofreTZK
Copy link
Collaborator Author

Avoid mutable fields on Block.t

ref: https://github.com/OnofreTZK/MoreOCaml/blob/main/NamedTuplesAndRecords/named_tuples_and_records.ml

@OnofreTZK OnofreTZK mentioned this issue May 24, 2023
Merged
@OnofreTZK OnofreTZK mentioned this issue Jun 1, 2023
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@OnofreTZK OnofreTZK

Labels
Sheldrake: improvement improve the code or infra Sheldrake: invalid business logic This doesn't seem right Sheldrake: question Further information is requested
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@OnofreTZK