Solution #1458
Solution #1458
Conversation
There was a problem hiding this comment.
Thank you for your hard work on implementing the cache decorator! 🎉 Your solution meets most of the task requirements and constraints, and it's great to see that it correctly handles caching and prints the expected messages. However, there's a critical issue with the return type of the wrapper function. It's currently specified as None, which is incorrect since the function is expected to return a value. Please update the return type to match the expected return type of the decorated function to ensure the decorator functions properly. Once this is addressed, your solution will be ready for approval. Keep up the great work! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
🔄 Fix the issues, commit the changes, and then re-request my review using the Re-request review button in the Reviewers tab.
| pass | ||
| data = {} | ||
|
|
||
| def wrapper(*args) -> None: |
There was a problem hiding this comment.
The return type of the wrapper function is specified as None, which is incorrect. The wrapper function should return the result of the func call, so the return type should match the return type of func. Consider changing the return type to match the expected return type of the decorated function.
No description provided.