-
-
Notifications
You must be signed in to change notification settings - Fork 130
Update find-factorial.md #110
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
Conversation
Use Python's built-in factorial function
β Deploy Preview for quicksnip ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
As this now just wraps over a standard library function without additional functionnality, I think we are safe to delete this snippet. Feel free to push a commit deleting it to your PR we will then merge
Should this be done for other lesser-known Python functionality too?
For instance:
https://github.com/dostonnabotov/quicksnip/blob/main/snippets/python/string-manipulation/reverse-string.md
Anything that has a one std function alternative shouldn't be here, but i don't think reversing a string has one ?
Closing this as stale.
If you want to contribute, please create a new PR following our guidelines.
Thanks
Use Python's built-in factorial function
Description
Type of Change
Checklist
Related Issues
Closes #
Additional Context
Screenshots (Optional)
Click to view screenshots