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

Exercise hash_exercises.rb: rename favorite_list to favorite_hash for clarity #97

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sometimescasey
Copy link

Because

favorite_hash is a less ambiguous parameter name

This PR

  • renames all instances of favorite_list to favorite_hash in hash_exercises.rb
  • and analogously, renames original_list and additional_list

Issue

Closes #96

Additional Information

Pull Request Requirements

  • I have thoroughly read and understand The Odin Project Contributing Guide
  • The title of this PR follows the location of change: brief description of change format, e.g. Data types exercise: Update spec files
  • The Because section summarizes the reason for this PR
  • The This PR section has a bullet point list describing the changes in this PR
  • If this PR addresses an open issue, it is linked in the Issue section
  • If this PR includes changes that needs to be updated on the solutions branch, I have created another PR (and linked it to this PR).

@JoshDevHub JoshDevHub self-requested a review February 14, 2024 21:26
@JoshDevHub JoshDevHub self-assigned this Feb 14, 2024
Copy link
Contributor

@JoshDevHub JoshDevHub left a comment

Choose a reason for hiding this comment

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

This is a nice change. Thanks for making it. Definitely agree that using hash makes things a bit more clear, especially for people who may have some familiarity with languages that have a List type (python, C#, and so on).

Sorry it took several weeks for us to get around to reviewing it.

One quick thing I wanted to ask @sometimescasey : this repo has a solutions branch that holds the answers to all of the exercises. You can view it here. Would you be okay making another quick PR that makes the same changes to that branch?

If you don't want to, that's okay and we can put someone else on it. Just let me know.

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.

Bug - Exercisehash_exercises.rb:favorite_list should be renamed to favorite_hash
2 participants