You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tooling.apache.org by wa...@apache.org on 2025/11/24 18:57:35 UTC
(tooling-trusted-releases) branch fstring-convetion created (now 6a44daf)
This is an automated email from the ASF dual-hosted git repository.
wave pushed a change to branch fstring-convetion
in repository https://gitbox.apache.org/repos/asf/tooling-trusted-releases.git
at 6a44daf Add note about f-string code convetion
This branch includes the following new commits:
new 6a44daf Add note about f-string code convetion
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@tooling.apache.org
For additional commands, e-mail: commits-help@tooling.apache.org
(tooling-trusted-releases) 01/01: Add note about f-string code convetion
Posted by wa...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
wave pushed a commit to branch fstring-convetion
in repository https://gitbox.apache.org/repos/asf/tooling-trusted-releases.git
commit 6a44daf71ca7a286cd124534cd7b74bb654132fd
Author: Dave Fisher <da...@davefisher.tech>
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: commits-unsubscribe@tooling.apache.org
For additional commands, e-mail: commits-help@tooling.apache.org