-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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] Numpy - split() #6437
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 @karishma-battina the entry looks good! 🚀
However, it is missing the ##Codebyte Example block. So, could you please add the ## Codebyte Example block at the end. Please note that the codebyte section is not followed by an output block, it just showcases the term in use.
Thank you!
@mamtawardhani, Thanks for the review. I have added the codebyte example block. |
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.
The entry looks good for a second round of 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.
The entry looks good to be merged, @karishma-battina! 🚀
👋 @karishma-battina 🎉 Your contribution(s) can be seen here: https://www.codecademy.com/resources/docs/numpy/built-in-functions/split Please note it may take a little while for changes to become visible. |
Description
Introduce a new term entry on the .split() under built-in functions in NumPy.
Created file: docs/content/numpy/concepts/built-in-functions/terms/split/split.md
Issue Solved
Closes #6421
Type of Change
New Term for Existing Concept Entry
Checklist
main
branch.Issues Solved
section.