-
Notifications
You must be signed in to change notification settings - Fork 4.3k
[Term Entry] Python functools: update_wrapper() #7481
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
[Term Entry] Python functools: update_wrapper() #7481
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.
Looks good for a second review! 🚀
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.
LGTM 🚀
|
@jjwheatley never use |
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 @jjwheatley, thanks so much for this contribution! The changes look good. Just a friendly heads-up for future branches: it's a good practice to avoid using special characters like parentheses () in branch names.
The main reason is that they have special meaning on the command line, which forces us to wrap the branch name in quotes for every single git operation (fetch, checkout, push, etc.). They can also cause issues with URL encoding and other development tools.
Using hyphens to separate words (e.g., update-functools-wrapper) is a common convention that avoids these problems.
|
👋 @jjwheatley 🎉 Your contribution(s) can be seen here: https://www.codecademy.com/resources/docs/python/functools-module/update-wrapper Please note it may take a little while for changes to become visible. |
Description
Added Term-Entry for Python's update_wrapper() method from the functools module.
Issue Solved
Closes #7456
Type of Change
Adding a new entry
Updating the documentation
Checklist
mainbranch.Issues Solvedsection.