Skip to content

Tweaks to string-formatting concept#4099

Open
BNAndras wants to merge 8 commits intoexercism:mainfrom
BNAndras:update-string-formatting
Open

Tweaks to string-formatting concept#4099
BNAndras wants to merge 8 commits intoexercism:mainfrom
BNAndras:update-string-formatting

Conversation

@BNAndras
Copy link
Member

No description provided.

@BethanyG
Copy link
Member

You should add yourself to the contributors list here and the links doc might need an update or once-over. 🙂 More comments as I finish up my coffee.

Copy link
Member

@BethanyG BethanyG left a comment

Choose a reason for hiding this comment

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

This largely looks good (nice work!). I might have more adjustments as we get the exercise going, but for now the changes are good and my suggestions are minimal.

Still noodling on if we want the related exercise to cover choosing between the methods, or if we only want to focus on f-strings. My first impulse is to do a mix of str.format() and f-strings, and leave the other stuff out. I worry that although template strings are important, they will confuse students, especially if/when we introduce Python 3.14s T-strings.

@BNAndras
Copy link
Member Author

BNAndras commented Mar 2, 2026

I'll take a closer look after work about adding an example for escapes not being applied and the formatting / alignment examples as well.

Copy link
Member

@BethanyG BethanyG left a comment

Choose a reason for hiding this comment

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

I like this version! Just a few small nits, but nice work overall. 😄

BNAndras and others added 3 commits March 13, 2026 19:03
Co-authored-by: BethanyG <BethanyG@users.noreply.github.com>
Co-authored-by: BethanyG <BethanyG@users.noreply.github.com>
Co-authored-by: BethanyG <BethanyG@users.noreply.github.com>
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