FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix minor unreachable code caused by t.Fatal by Abirdcfly · Pull Request #6283 · letsencrypt/boulder · GitHub

fix minor unreachable code caused by t.Fatal - #6283

Merged
aarongable merged 1 commit into
letsencrypt:mainfrom
Abirdcfly:patch-1
Aug 12, 2022
Merged

aarongable merged 1 commit into
letsencrypt:mainfrom
Abirdcfly:patch-1

Conversation

Copy link
Copy Markdown
Contributor

Signed-off-by: Abirdcfly fp544037857@gmail.com

Abirdcfly requested a review from a team as a code owner August 10, 2022 07:48

aarongable left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Good catch! Rather than deleting this line, please move it above the t.Fatal() call.

Comment thread reloader/reloader_test.go

Copy link
Copy Markdown
Contributor Author

Good catch! Rather than deleting this line, please move it above the t.Fatal() call.

Done. Thanks for review.

Abirdcfly changed the title delete minor unreachable code caused by t.Fatal fix minor unreachable code caused by t.Fatal Aug 11, 2022
Signed-off-by: Abirdcfly <fp544037857@gmail.com>
aarongable merged commit 7cd35f4 into letsencrypt:main Aug 12, 2022
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL