Skip to content

Conversation

@IsaacG
Copy link
Member

@IsaacG IsaacG commented Jun 3, 2025

No description provided.

@IsaacG IsaacG requested a review from BethanyG June 3, 2025 04:02
@github-actions github-actions bot added track/python Python track type/mentor-notes Mentor notes labels Jun 3, 2025
BethanyG
BethanyG previously approved these changes Jun 3, 2025
Copy link
Member

@BethanyG BethanyG left a comment

Choose a reason for hiding this comment

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

Nice work! Thanks for writing these up.

One niggle: There is a 5th "task", which is adding docstrings to the functions, which we might want to add a sentence on, with ref to PEP257 (and if you wanted to, sample feedback).

But its a niggle, so I am marking this approved, and you can merge or add the comments as you see fit. 😄

Copy link
Member

@BethanyG BethanyG left a comment

Choose a reason for hiding this comment

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

Awesome! Thanks for adding docstrings in. Good to merge from my side. 😄

@IsaacG IsaacG merged commit ab496ac into exercism:main Jun 3, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

track/python Python track type/mentor-notes Mentor notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants