Skip to content
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

Snowman project scaffold updates for C23 #3

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

kelsey-steven-ada
Copy link

Part of Asana task Change Snowman Submission to Learn

  • Changes snowman image from a series of variables over to a list and uses the list-based printing function we show in Learn.
  • Splits print_word_progress_string into 2 functions, print_word_progress_string and generate_word_progress_string
  • Renames build_word_dict to build_letter_status_dict
  • Renames get_word_progress to is_word_guessed
  • Renames snowman_word_dict to correct_letter_guess_statuses

…graphic and printing to list-based version shown in lists lesson. use same parameter name as in lessons for print_snowman_graphic
…g and print_word_progress_string to separate concerns
… snowman_word_dict over to correct_letter_guess_statuses. Small changes to a couple functions' structure to better match Learn and edge cases like no extra space at the end of the word progress string
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.

1 participant