Skip to content

[Roman Numerals] Approach docs improvements - #4300

Draft
Yrahcaz7 wants to merge 2 commits into
exercism:mainfrom
Yrahcaz7:roman-numerals-approach-improvements
Draft

Yrahcaz7 wants to merge 2 commits into
exercism:mainfrom
Yrahcaz7:roman-numerals-approach-improvements

Conversation

@Yrahcaz7

Copy link
Copy Markdown
Contributor

This PR covers all of the remaining audit items listed in #4289 (see issue 4197 for the original discussion). It also adds a few links to the Python docs where relevant.

However, this PR is still a draft because the recursion approach needs more explanation, and the itertools.starmap() approach would likely benefit from reorganization (these two items were not in the original audit list).

@BethanyG

Copy link
Copy Markdown
Member

@Yrahcaz7 - Thank you so much for this!

However, this PR is still a draft because the recursion approach needs more explanation, and the itertools.starmap() approach would likely benefit from reorganization (these two items were not in the original audit list).

Happy to have you add those two things here, if you'd like. We may as well change whatever needs changing, since we are not likely to not this way again soon. 🙂

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