Skip to content

Add baffling-birthdays exercice#1004

Merged
mk-mxp merged 6 commits into
exercism:mainfrom
resu-xuniL:baffling-birthdays
Jun 28, 2026
Merged

Add baffling-birthdays exercice#1004
mk-mxp merged 6 commits into
exercism:mainfrom
resu-xuniL:baffling-birthdays

Conversation

@resu-xuniL

Copy link
Copy Markdown
Contributor

Another exercice for validation.

@resu-xuniL resu-xuniL force-pushed the baffling-birthdays branch from 2f0c983 to e1371c1 Compare June 10, 2026 12:41

@mk-mxp mk-mxp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great solution! I'm always a bit sceptical about testing generated random results for complete coverage (e.g. 31 days within 1000 tries). But maybe that's me being too pedantic.

Comment thread exercises/practice/baffling-birthdays/BafflingBirthdaysTest.php Outdated
Comment thread exercises/practice/baffling-birthdays/BafflingBirthdaysTest.php Outdated
Comment thread exercises/practice/baffling-birthdays/BafflingBirthdaysTest.php Outdated
Comment thread exercises/practice/baffling-birthdays/BafflingBirthdaysTest.php Outdated
Comment thread exercises/practice/baffling-birthdays/BafflingBirthdaysTest.php Outdated
Comment thread exercises/practice/baffling-birthdays/BafflingBirthdaysTest.php Outdated
Comment thread exercises/practice/baffling-birthdays/BafflingBirthdaysTest.php Outdated
Comment thread exercises/practice/baffling-birthdays/BafflingBirthdaysTest.php Outdated
Comment thread exercises/practice/baffling-birthdays/BafflingBirthdaysTest.php
Comment thread config.json Outdated

@mk-mxp mk-mxp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Test run well below one second, that's fast enough. Thanks a lot for your contribution!

@mk-mxp mk-mxp added x:action/create Work on something from scratch x:knowledge/intermediate Quite a bit of Exercism knowledge required x:module/practice-exercise Work on Practice Exercises x:type/content Work on content (e.g. exercises, concepts) x:size/medium Medium amount of work x:rep/medium Medium amount of reputation labels Jun 28, 2026
@mk-mxp mk-mxp merged commit 855f886 into exercism:main Jun 28, 2026
13 checks passed
@resu-xuniL

Copy link
Copy Markdown
Contributor Author

thanks for your help!

@resu-xuniL resu-xuniL deleted the baffling-birthdays branch June 28, 2026 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

x:action/create Work on something from scratch x:knowledge/intermediate Quite a bit of Exercism knowledge required x:module/practice-exercise Work on Practice Exercises x:rep/medium Medium amount of reputation x:size/medium Medium amount of work x:type/content Work on content (e.g. exercises, concepts)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants