Skip to content
This repository has been archived by the owner on Sep 14, 2023. It is now read-only.

chore: misc cleanup #672

Merged
merged 7 commits into from
Feb 28, 2023
Merged

chore: misc cleanup #672

merged 7 commits into from
Feb 28, 2023

Conversation

harrysolovay
Copy link
Contributor

  • get rid of PromiseOr
  • move crypto utils into dedicated dir
  • remove unnecessary branded types

@harrysolovay harrysolovay requested a review from tjjfvi as a code owner February 27, 2023 22:10
tjjfvi
tjjfvi reviewed Feb 27, 2023
View reviewed changes
patterns/consensus/babeBlockAuthor.ts Outdated Show resolved Hide resolved
patterns/consensus/preRuntimeDigest.ts Outdated Show resolved Hide resolved
util/mod.ts Outdated Show resolved Hide resolved
@harrysolovay harrysolovay requested a review from tjjfvi February 28, 2023 13:00
@harrysolovay harrysolovay merged commit 21a4100 into main Feb 28, 2023
@harrysolovay harrysolovay deleted the get-rid-of-promise-or branch February 28, 2023 15:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@tjjfvi tjjfvi tjjfvi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@harrysolovay @tjjfvi