-
Notifications
You must be signed in to change notification settings - Fork 736
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.
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
[Merged by Bors] - Cleanup unreachable code in lcli::generate_bootnode_enr
and some tests
#4485
[Merged by Bors] - Cleanup unreachable code in lcli::generate_bootnode_enr
and some tests
#4485
Conversation
@michaelsproul / @pawanjay176 🙏 can I have your review? fairly low hanging 🍒 since age is away |
bors r+ |
🕐 Waiting for PR status (GitHub check) to be set, probably by CI. Bors will automatically try to run when all required PR statuses are set. |
I'll merge unstable once #4503 gets in |
Canceled. |
bors r+ |
…sts (#4485) ## Issue Addressed n/a Noticed this while working on something else ## Proposed Changes - leverage the appropriate types to avoid a bunch of `unwrap` and errors ## Additional Info n/a
Pull request successfully merged into unstable. Build succeeded! The publicly hosted instance of bors-ng is deprecated and will go away soon. If you want to self-host your own instance, instructions are here. If you want to switch to GitHub's built-in merge queue, visit their help page.
|
lcli::generate_bootnode_enr
and some testslcli::generate_bootnode_enr
and some tests
…sts (sigp#4485) ## Issue Addressed n/a Noticed this while working on something else ## Proposed Changes - leverage the appropriate types to avoid a bunch of `unwrap` and errors ## Additional Info n/a
…sts (sigp#4485) ## Issue Addressed n/a Noticed this while working on something else ## Proposed Changes - leverage the appropriate types to avoid a bunch of `unwrap` and errors ## Additional Info n/a
Issue Addressed
n/a Noticed this while working on something else
Proposed Changes
unwrap
and errorsAdditional Info
n/a