-
Notifications
You must be signed in to change notification settings - Fork 105
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
feat: Add Piscina#close
API
#396
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey! Sorry for the fast review but it kept my attention as I was working on the same feature 😅
99f11da
to
08e995b
Compare
4909721
to
41233ca
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work! Minor comments left 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work 🚀
Small nit
and we are ready to go
Hi everyone!
This PR adds a new method called
close()
. It allows you to stop all the workers in a pool gracefully (closes: #139)Please let me know if you have any feedback or suggestions for improvement. I'm happy to make any changes if needed.