-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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] Dart Maps .remove() #4420
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 @dancikmad, I noticed that you have mentioned list.remove()
method in the content, please change it to map.remove()
. Ping me when you are done with it.
- 'paths/computer-science' | ||
--- | ||
|
||
In Dart, the **`.remove()`** method is used to remove the first occurence of a specified value from the list. This method is useful when a particular element from a list has to be removed without knowing it's index. It returns true if the specified value is removed from the list. |
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.
This entry is for Map concept so you need to write the content in reference to Maps. You have mentioned lists everywhere in the content, please replace it.
@avdhoottt Hi, yeah I know. My mistake ... I realised this morning after I went through the doc again. |
Hey @dancikmad Are you working on this PR? |
@avdhoottt Yes, I have redone it, I will push the change later this evening. Sorry for the delay, been busy with a side project this week. |
Hey @dancikmad, I think you have mistakenly committed |
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 @dancikmad, please make these final changes!!
map_name.remove(key) | ||
``` | ||
|
||
- `key`: The key of the entry to be removed from the map. |
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.
This line contains plagiarism, please change it.
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 the 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.
Looks good! 👍
👋 @dancikmad 🎉 Your contribution(s) can be seen here: https://www.codecademy.com/resources/docs/dart/map/remove Please note it may take a little while for changes to become visible. |
Description
Issue Solved
Type of Change
Checklist
main
branch.Issues Solved
section.