Skip to content

fix: clarify note about reverting Countdown function #858

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

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

Conversation

yemiwebby
Copy link

@yemiwebby yemiwebby commented Apr 22, 2025

While following the mocking chapter of the book, I ran into a test failure after modifying the Countdown() function as suggested. The issue was that there wasn't an explicit instruction to revert the code snippet to its original, working version afterwards.

This small detail can easily confuse Go beginners who are following along line by line. Although the source code serves as a reference, adding a clear note in the book helps avoid unnecessary confusion and improves the learning experience.

This PR adds that clarification as a note just after the line:

if you run your tests they should still be passing even though the implementation is wrong.

@quii

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