Skip to content

Add baffling-birthdays exercise - #236

Open
ageron wants to merge 9 commits into
exercism:mainfrom
ageron:add-baffling-birthdays
Open

Add baffling-birthdays exercise#236
ageron wants to merge 9 commits into
exercism:mainfrom
ageron:add-baffling-birthdays

Conversation

@ageron

@ageron ageron commented Sep 25, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@ageron ageron added the ready-to-translate Ready to translate label Sep 25, 2026
@exercism-i18n exercism-i18n Bot mentioned this pull request Sep 25, 2026
Closed
exercism-i18n Bot added a commit to exercism/i18n that referenced this pull request Sep 25, 2026
@exercism-i18n @claude
Translate exercism/roc#236: 12 item(s)
256e559 
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@exercism-i18n

exercism-i18n Bot commented Sep 25, 2026

Copy link
Copy Markdown

This PR has been translated 🚀

@ageron ageron added ready-to-translate Ready to translate and removed ready-to-translate Ready to translate labels Sep 26, 2026
@exercism-i18n exercism-i18n Bot mentioned this pull request Sep 26, 2026
Closed
exercism-i18n Bot added a commit to exercism/i18n that referenced this pull request Sep 26, 2026
@exercism-i18n @claude
Translate exercism/roc#236: 15 item(s)
b888609 
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@exercism-i18n

exercism-i18n Bot commented Sep 26, 2026

Copy link
Copy Markdown

This PR has been translated 🚀

@github-actions github-actions Bot removed the ready-to-translate Ready to translate label Sep 27, 2026
@github-actions

github-actions Bot commented Sep 27, 2026

Copy link
Copy Markdown
Contributor

This push changed English, so ready-to-translate has come off and any queued translation is withdrawn.

A maintainer: please re-apply ready-to-translate once the copy is final. That queues the translation at the PR's head commit.

@ageron ageron added the ready-to-translate Ready to translate label Sep 27, 2026
@exercism-i18n exercism-i18n Bot mentioned this pull request Sep 27, 2026
Closed
exercism-i18n Bot added a commit to exercism/i18n that referenced this pull request Sep 27, 2026
@exercism-i18n @claude
Translate exercism/roc#236: 9 item(s)
71bbcaf 
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@exercism-i18n

exercism-i18n Bot commented Sep 27, 2026

Copy link
Copy Markdown

This PR has been translated 🚀

@github-actions github-actions Bot removed the ready-to-translate Ready to translate label Oct 4, 2026
@github-actions

github-actions Bot commented Oct 4, 2026

Copy link
Copy Markdown
Contributor

This push changed English, so ready-to-translate has come off and any queued translation is withdrawn.

A maintainer: please re-apply ready-to-translate once the copy is final. That queues the translation at the PR's head commit.

@ageron ageron added the ready-to-translate Ready to translate label Oct 4, 2026
@exercism-i18n exercism-i18n Bot mentioned this pull request Oct 4, 2026
Closed
exercism-i18n Bot added a commit to exercism/i18n that referenced this pull request Oct 4, 2026
@exercism-i18n @claude
Translate exercism/roc#236: 36 item(s)
616533a 
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@exercism-i18n

exercism-i18n Bot commented Oct 4, 2026

Copy link
Copy Markdown

This PR has been translated 🚀

@github-actions github-actions Bot removed the ready-to-translate Ready to translate label Oct 6, 2026
@github-actions

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

This push changed English, so ready-to-translate has come off and any queued translation is withdrawn.

A maintainer: please re-apply ready-to-translate once the copy is final. That queues the translation at the PR's head commit.

@ageron ageron added the ready-to-translate Ready to translate label Oct 25, 2026
@exercism-i18n exercism-i18n Bot mentioned this pull request Oct 25, 2026
Closed
exercism-i18n Bot added a commit to exercism/i18n that referenced this pull request Oct 25, 2026
@exercism-i18n @claude
Translate exercism/roc#236: 18 item(s)
bcf19ef 
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@exercism-i18n

exercism-i18n Bot commented Oct 25, 2026

Copy link
Copy Markdown

This PR has been translated 🚀

Anton-4
Anton-4 reviewed Oct 26, 2026
View reviewed changes
Comment thread exercises/practice/baffling-birthdays/.docs/instructions.append.md
```

This code implicitly calls `DateOfBirth.from_quote("2019-01-20")` at compile time.
This can potentially speed up the program since it doesn't need to handle parsing at runtime, and it ensures that dates rejected by `from_quote` are caught at compile time, rather than at runtime.

@Anton-4 Anton-4 Oct 26, 2026 •
edited
Loading

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.

This makes it sound like this is specific to from_quote. It is explained later but I suspect people may easily misremember if they are presented the info like this.

@ageron ageron Oct 26, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ah good point. How about:

This code implicitly calls `DateOfBirth.from_quote("2019-01-20")` at compile time.
Running code at compile time using this approach (or other methods described below) can
potentially speed up the program since it doesn't need to handle parsing and data validation
at runtime. Moreover, it ensures that invalid data (such as an incorrect date format) is rejected
at compile time rather than at runtime, making your app more robust.

@Anton-4 Anton-4 Oct 27, 2026

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.

That looks good :)
Can you apply this for other examples too that have a similar from_quote explanation (merged and unmerged)?

@github-actions github-actions Bot removed the ready-to-translate Ready to translate label Oct 27, 2026
@github-actions

github-actions Bot commented Oct 27, 2026

Copy link
Copy Markdown
Contributor

This push changed English, so ready-to-translate has come off and any queued translation is withdrawn.

A maintainer: please re-apply ready-to-translate once the copy is final. That queues the translation at the PR's head commit.

@ageron ageron added the ready-to-translate Ready to translate label Oct 27, 2026
@exercism-i18n exercism-i18n Bot mentioned this pull request Oct 27, 2026
Closed
exercism-i18n Bot added a commit to exercism/i18n that referenced this pull request Oct 27, 2026
@exercism-i18n @claude
Translate exercism/roc#236: 36 item(s)
7d1e8f4 
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@exercism-i18n

exercism-i18n Bot commented Oct 27, 2026

Copy link
Copy Markdown

This PR has been translated 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Anton-4 Anton-4 Anton-4 requested changes

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

ready-to-translate Ready to translate

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ageron @Anton-4