Skip to content

Use Timeout.InfiniteTimeSpan#668

Merged
bitfaster merged 1 commit into
bitfaster:mainfrom
Joy-less:use-timeout.infinitetimespan
Feb 13, 2025
Merged

Use Timeout.InfiniteTimeSpan#668
bitfaster merged 1 commit into
bitfaster:mainfrom
Joy-less:use-timeout.infinitetimespan

Conversation

@Joy-less

@Joy-less Joy-less commented Feb 5, 2025

Copy link
Copy Markdown
Contributor

This is just a readability thing. Use Timeout.InfiniteTimeSpan not new TimeSpan(0, 0, 0, 0, -1).

@coveralls

coveralls commented Feb 6, 2025

Copy link
Copy Markdown

Coverage Status

coverage: 99.169% (+0.02%) from 99.152%
when pulling 3c25cfc on Joy-less:use-timeout.infinitetimespan
into da00197 on bitfaster:main.

bitfaster
bitfaster approved these changes Feb 6, 2025
View reviewed changes

@bitfaster bitfaster left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGTM

@Joy-less Joy-less mentioned this pull request Feb 6, 2025
Open
@bitfaster bitfaster merged commit 0c6dd7f into bitfaster:main Feb 13, 2025
@Joy-less Joy-less deleted the use-timeout.infinitetimespan branch February 13, 2025 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bitfaster bitfaster bitfaster 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.

3 participants

@Joy-less @coveralls @bitfaster