This is an automated email from the ASF dual-hosted git repository.
sbp pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tooling-trusted-releases.git
The following commit(s) were added to refs/heads/main by this push:
new 161cb5f Add note about f-string code convetion
161cb5f is described below
commit 161cb5fdd2b1b5bd8bdd5d89da8589fef3b2447c
Author: Dave Fisher <[email protected]>
AuthorDate: Mon Nov 24 10:56:34 2025 -0800
Add note about f-string code convetion
---
atr/docs/code-conventions.md | 2 ++
1 file changed, 2 insertions(+)
diff --git a/atr/docs/code-conventions.md b/atr/docs/code-conventions.md
index 4bd987b..edae328 100644
--- a/atr/docs/code-conventions.md
+++ b/atr/docs/code-conventions.md
@@ -251,6 +251,8 @@ Using `case _` breaks type checking in such situations.
This should be adhered to even in contexts where printf style is usually
expected, such as in `log.info` calls, unless there is a reason not to, such as
when there are specific printf style flags which have no f-string equivalent.
+> While this choice appears arbitrary it is a preference which is not enforced
by any checks. Enforcement is via code review. See [issue
#339](https://github.com/apache/tooling-trusted-releases/issues/339) for a
discussion.
+
## HTML
### Use sentence case for headings, form labels, and submission buttons
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
