Skip to content

fix(curriculum): update print tests for build a bill splitter steps 7-8#67831

Merged
majestic-owl448 merged 1 commit into
freeCodeCamp:mainfrom
AbiramiR-27:fix/curriculum-bill-splitter-print-tests
Jun 8, 2026
Merged

fix(curriculum): update print tests for build a bill splitter steps 7-8#67831
majestic-owl448 merged 1 commit into
freeCodeCamp:mainfrom
AbiramiR-27:fix/curriculum-bill-splitter-print-tests

Conversation

@AbiramiR-27
Copy link
Copy Markdown
Contributor

Checklist:

Closes #67819

Updates Steps 6, 7, and 8 of the "Build a Bill Splitter" Python project to use stdout redirection and regex validation for checking print outputs (with a float tolerance of 1e-6). This aligns them with Steps 4 and 5, allowing valid alternative print formats (like string concatenation) that were previously blocked by structural AST syntax checking.

@AbiramiR-27 AbiramiR-27 requested review from a team as code owners June 5, 2026 18:04
@github-actions github-actions Bot added the scope: curriculum Lessons, Challenges, Projects and other Curricular Content in curriculum directory. label Jun 5, 2026
Copy link
Copy Markdown
Contributor

@majestic-owl448 majestic-owl448 left a comment

Choose a reason for hiding this comment

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

a pr that updates step 6 has just been merged, if you remove changes from that file in this PR we can review the changes to step 7 and 8

@AbiramiR-27
Copy link
Copy Markdown
Contributor Author

Thanks for letting me know. I'll remove the Step 6 changes from this PR and update it so only the Step 7 and Step 8 changes remain.

@AbiramiR-27 AbiramiR-27 force-pushed the fix/curriculum-bill-splitter-print-tests branch from 9886b60 to c91b67e Compare June 7, 2026 05:57
@majestic-owl448 majestic-owl448 changed the title fix(curriculum): update print tests for build a bill splitter steps 6-8 fix(curriculum): update print tests for build a bill splitter steps 7-8 Jun 7, 2026
Copy link
Copy Markdown
Contributor

@majestic-owl448 majestic-owl448 left a comment

Choose a reason for hiding this comment

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

Hi @username. Congrats on your first pull request (PR)! 🎉

Thank you for your contribution to the page! 👍
We are happy to accept these changes and look forward to future contributions. 📝

@majestic-owl448 majestic-owl448 enabled auto-merge (squash) June 8, 2026 07:03
@majestic-owl448 majestic-owl448 merged commit 057a787 into freeCodeCamp:main Jun 8, 2026
16 checks passed
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jun 8, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

scope: curriculum Lessons, Challenges, Projects and other Curricular Content in curriculum directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bill splitter step 6

2 participants