Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prefer format or f-strings over percent formatting #109

Merged
merged 4 commits into from
Dec 5, 2024

Conversation

bbannier
Copy link
Member

@bbannier bbannier commented Dec 4, 2024

No description provided.

@bbannier bbannier self-assigned this Dec 4, 2024
@bbannier bbannier changed the title Prefer format of f-strings over percent formattingw Prefer format of f-strings over percent formatting Dec 4, 2024
@bbannier bbannier changed the title Prefer format of f-strings over percent formatting Prefer format or f-strings over percent formatting Dec 4, 2024
@bbannier bbannier force-pushed the topic/bbannier/upgrade-string-formatting branch 2 times, most recently from fa6f354 to e7e8dcf Compare December 4, 2024 08:40
@bbannier bbannier marked this pull request as ready for review December 4, 2024 08:44
@bbannier bbannier requested a review from timwoj December 4, 2024 08:44
@timwoj
Copy link
Member

timwoj commented Dec 4, 2024

Some of these might have been caught by #108 already. Let me merge that and you can rebase this one.

.github/workflows/btest.yml Outdated Show resolved Hide resolved
btest Outdated Show resolved Hide resolved
@bbannier bbannier force-pushed the topic/bbannier/upgrade-string-formatting branch from e7e8dcf to b011560 Compare December 5, 2024 12:15
@bbannier bbannier force-pushed the topic/bbannier/upgrade-string-formatting branch from b011560 to 51d5cd8 Compare December 5, 2024 12:17
@bbannier bbannier requested a review from timwoj December 5, 2024 13:04
@timwoj timwoj merged commit a939b12 into master Dec 5, 2024
22 checks passed
@timwoj timwoj deleted the topic/bbannier/upgrade-string-formatting branch December 5, 2024 19:38
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