-
Notifications
You must be signed in to change notification settings - Fork 4.1k
Moved bulk of Python: Functions info into separate entries #701
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
Moved bulk of Python: Functions info into separate entries #701
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.
@Dusch4593, I reviewed this and left a few comments.
content/python/concepts/functions/terms/anonymous-functions/anonymous-functions.md
Outdated
Show resolved
Hide resolved
content/python/concepts/functions/terms/parameters/parameters.md
Outdated
Show resolved
Hide resolved
content/python/concepts/functions/terms/parameters/parameters.md
Outdated
Show resolved
Hide resolved
Co-authored-by: SSwiniarski <86081858+SSwiniarski@users.noreply.github.com>
…onymous-functions.md Co-authored-by: SSwiniarski <86081858+SSwiniarski@users.noreply.github.com>
Co-authored-by: SSwiniarski <86081858+SSwiniarski@users.noreply.github.com>
@SSwiniarski I'll incorporate the rest of your suggestions locally and push from there. 😄 |
@SSwiniarski Finished with incorporating your suggestions. |
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.
@Dusch4593 LGTM! 👍
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.
@Dusch4593 I've added some feedback to this.
content/python/concepts/functions/terms/arguments-parameters/arguments-parameters.md
Outdated
Show resolved
Hide resolved
content/python/concepts/functions/terms/arguments-parameters/arguments-parameters.md
Outdated
Show resolved
Hide resolved
Co-authored-by: KTom101 <kyrathompson@me.com>
Co-authored-by: KTom101 <kyrathompson@me.com>
Thanks for your feedback @KTom101 🙏🏻 I'm going to implement the rest of it locally and push from there 😃 |
@KTom101 I believe all of the comments/suggestions were addressed? |
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.
@Dusch4593 Yep, they were. Thanks!
Thank you everyone! @KTom101 @SSwiniarski! The edits and entries will be shown below.
|
Description
Per our target keyword efforts.
Type of Change
Checklist
main
branch.