Skip to content
This repository was archived by the owner on Jul 10, 2025. It is now read-only.

Conversation

@coder11
Copy link
Contributor

@coder11 coder11 commented Jul 18, 2022

No description provided.

@coder11 coder11 requested a review from shamsartem July 19, 2022 00:15
@coder11 coder11 requested a review from shamsartem August 4, 2022 14:43
@coder11 coder11 requested a review from shamsartem August 4, 2022 16:31
Copy link
Contributor

@shamsartem shamsartem left a comment

Choose a reason for hiding this comment

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

Please fix this small issues before merging. Thanks!

Comment on lines 422 to 424
if (!config.KeyPair) {
throw new Error('Key Pair is required');
}
Copy link
Contributor

Choose a reason for hiding this comment

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

When you added the type & Required<Pick<PeerConfig, 'KeyPair'>> above you removed the necessety for this run time if (!config.KeyPair) check. Please remove it

@coder11 coder11 merged commit fb50ec2 into master Aug 5, 2022
@coder11 coder11 deleted the ephemeral-networks branch August 5, 2022 13:43
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.

3 participants