Skip to content

Added dict#3650

Open
sarychev90 wants to merge 1 commit into
mate-academy:masterfrom
sarychev90:master
Open

Added dict#3650
sarychev90 wants to merge 1 commit into
mate-academy:masterfrom
sarychev90:master

Conversation

@sarychev90

Copy link
Copy Markdown

Added dict

@mateacademy-ai-mentor mateacademy-ai-mentor left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review Summary

Your solution is approved because it meets all the core requirements of the task. You defined the 8 required variables in app/main.py and correctly created the sorted_variables dictionary with the keys "mutable" and "immutable". Each list contains the appropriate types: mutable containers like lists, dictionaries, and sets are grouped under "mutable", while numbers, booleans, strings, and tuples are under "immutable". All 8 variables are present in one of the two lists, satisfying the checklist items about completeness and structure.

One conceptual point to keep in mind: set is indeed a mutable type in Python, so including your collection_of_coins (the set) in the "mutable" list is correct and matches standard behavior. No functional or structural changes are needed, but continuing to reinforce the difference between mutable and immutable types will help you in future tasks. Nicely done on following the task instructions closely and organizing the data types correctly.


✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨

Footnotes

  1. Rate AI review example

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants