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

progress: Don't leak time.Tick channel #152

Closed
wants to merge 1 commit into from
Closed

progress: Don't leak time.Tick channel #152

wants to merge 1 commit into from

Conversation

virtuald
Copy link
Contributor

@virtuald virtuald commented Apr 4, 2021

See title.

@jedib0t
Copy link
Owner

jedib0t commented Apr 6, 2021

Thanks for the changes!

jedib0t pushed a commit that referenced this pull request Apr 6, 2021
@virtuald @jedib0t
progress: Don't leak time.Tick channel; merging #152
6744030
@jedib0t
Copy link
Owner

jedib0t commented Apr 6, 2021

Closing this PR. Your commit is now merged manually.

@jedib0t jedib0t closed this Apr 6, 2021
@virtuald virtuald deleted the tick-leak branch April 6, 2021 06:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jedib0t jedib0t jedib0t 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.
2 participants
@virtuald @jedib0t