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

corrections to chapter 2 #22

Merged
merged 1 commit into from
Mar 5, 2025
Merged

Conversation

KatherineCox
Copy link
Contributor

I think I found a few mistakes in the key.

I'm working through the book myself (self-study), so double check my fixes.

For future reference, would you rather have each correction be a separate PR or is a few at a time okay?

@spamegg1
Copy link
Owner

spamegg1 commented Mar 5, 2025

Thanks for the fixes! ❤️
If they are a few small fixes, it's OK to have them in one PR.
If it changes a solution's logic, then it would be better as a separate PR.

@spamegg1
Copy link
Owner

spamegg1 commented Mar 5, 2025

I checked them all, thanks again! ❤️

@spamegg1 spamegg1 merged commit 1ef6137 into spamegg1:master Mar 5, 2025
2 checks passed
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