Skip to content

Conversation

@Baboushka
Copy link
Contributor

pull request

Add Baffling Birthdays exercise based on the problem specifications.

I set the exercise's difficulty to 6 and included the for-loops, lists, sets, randomness, datetime prerequisites.

The modifications passed:

  • ./gradlew test
  • ./gradlew check
  • ./gradlew compileStarterTestJava
  • bin/configlet lint (no errors for baffling-birthdays)

Notes:

  • I added an extra test to verify birthdates follow the YYYY-MM-DD format (ref. 1st comment canonical-data)
  • Randomness related tests use the average of multiple runs. I also added some tolerance to allow for small deviations (ref. 2nd+3rd comments canonical-data)

Closes #2931


Reviewer Resources:

Track Policies

Copy link
Member

@kahgoh kahgoh left a comment

Choose a reason for hiding this comment

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

Thanks for contributing @Baboushka! Comments for this PR 👇🏻

@Baboushka
Copy link
Contributor Author

Thank you for the review @kahgoh!
Your feedback is included in the latest commits: I updated to Gradle 9.2.1, removed comments and the extra test, and updated probability tests to match the C# approach.
The intermediary commit had indentation issues, that's corrected in the last commit.

@kahgoh kahgoh merged commit 36e31a6 into exercism:main Jan 9, 2026
7 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.

Add baffling-birthdays exercise

2 participants