Skip to content
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.

Sign up for GitHub

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

Jump to bottom

all: use timer instead of time.After in loops, to avoid memleaks #91

Merged
merged 1 commit into from
Jun 12, 2024
Merged

all: use timer instead of time.After in loops, to avoid memleaks #91

merged 1 commit into from
Jun 12, 2024

Conversation

ryanmorphl2
Copy link
Contributor

@ryanmorphl2 ryanmorphl2 requested a review from a team as a code owner June 12, 2024 03:44
@FletcherMan FletcherMan added the release/0.3.x indicates that needs to be cherry-pick to release/0.3.x branch label Jun 12, 2024
@FletcherMan FletcherMan merged commit b3fc060 into main Jun 12, 2024
@FletcherMan FletcherMan deleted the ryan/time_after_leak branch June 12, 2024 06:50
github-actions bot pushed a commit that referenced this pull request Jun 12, 2024
@ryanmorphl2 @web-flow
all: use timer instead of time.After in loops, to avoid memleaks (#91)
9423688
FletcherMan pushed a commit that referenced this pull request Jun 12, 2024
@github-actions @ryanmorphl2
all: use timer instead of time.After in loops, to avoid memleaks (#97)
8fba72b 
all: use timer instead of time.After in loops, to avoid memleaks (#91)

Co-authored-by: Ryan He <163962984+ryanmorphl2@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FletcherMan FletcherMan FletcherMan approved these changes

Assignees
No one assigned
Labels
release/0.3.x indicates that needs to be cherry-pick to release/0.3.x branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ryanmorphl2 @FletcherMan