Skip to content

Comments

testing: add explanatory commentary to solution#3062

Open
mgeisler wants to merge 2 commits intomainfrom
solution-testing
Open

testing: add explanatory commentary to solution#3062
mgeisler wants to merge 2 commits intomainfrom
solution-testing

Conversation

@mgeisler
Copy link
Collaborator

This commentary, written by Gemini, focuses on aspects of the solution that differ from the baseline languages (C/Java/Python), highlighting Rust-specific idioms and concepts.

This commentary, written by Gemini, focuses on aspects of the solution that differ from the baseline languages (C/Java/Python), highlighting Rust-specific idioms and concepts.
@mgeisler mgeisler enabled auto-merge (squash) January 31, 2026 12:09
- Note that `c.to_digit(10)` returns `Some(digit)` if `c` is a base-10 digit,
and `None` otherwise.
- The condition `digits >= 2 && sum % 10 == 0` ensures both requirements are
met.
Copy link
Collaborator

Choose a reason for hiding this comment

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

What's the principle behind moving these specific items into speaker notes?

Copy link
Collaborator

Choose a reason for hiding this comment

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

The same question applies to all other PRs, including those that I merged. Could you take another look?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Looking at this example, I interpret the difference to be that the Gemini saw this API detail about to_digit as not important enough to put into the main text.

The non-speaker note text is firmly about the solution: what was made. The background information about to_digit is, well, more background.

I can make a pass and have it take out non-essential stuff like this.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Alright, I asked Gemini what it had to say for itself 😄 It came back with

  • Main Text: Concise summary of the logical fixes required by the exercise.
  • Speaker Notes: Deep-dive into Rust-specific design (Unicode, Option ergonomics, and architectural choices) that provide value for experienced engineers without needing comparison to other ecosystems.

I had it update the text based on this, let me know what you think!

@mgeisler mgeisler force-pushed the solution-testing branch 2 times, most recently from dbb6993 to 1b1ae62 Compare February 14, 2026 09:41
- Remove redundant or overly simplistic explanations.
- Focus on Rust-specific idioms and design choices.
- Clean up formatting and technical depth.
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.

2 participants