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.

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

Implement ProgressBar::enable_steady_tick_with_hook for adding custom tick behaviors #679

Closed
wants to merge 1 commit into from

Conversation

chris-laplante
Copy link
Collaborator

See #578 for motivation

@djc
Copy link
Member

djc commented Dec 12, 2024

I don't know, this seems like a lot of complexity when we don't really have a good use case right now (the original reporter was able to resolve their issue in another way)...

@mostthingsweb
Copy link

mostthingsweb commented Dec 12, 2024

I don't know, this seems like a lot of complexity when we don't really have a good use case right now (the original reporter was able to resolve their issue in another way)...

We can close it for now then if you'd like. I don't feel strongly about it, and can always revisit it if a real need arises. (I'll do it myself when I log into my work account... on personal account for time being)

@chris-laplante
Copy link
Collaborator Author

Closing until someone expresses a need for something like this.

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