sascha/esphome
1
0
Fork 0
mirror of https://github.com/esphome/esphome.git synced 2025-11-04 00:51:49 +00:00
Code Issues Releases Wiki Activity

Merge branch 'bluetooth_namespace_cpp17' into integration

Browse Source
This commit is contained in:
J. Nick Koston
2025-07-22 19:41:52 -10:00
parent edeee42fd9 5b8ae6ed1a
commit bfae905a2b
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
script/ci-custom.py
View File

@@ -575,13 +575,15 @@ def lint_namespace(fname, content):
expected_name = re.match(
r"^esphome/components/([^/]+)/.*", fname.replace(os.path.sep, "/")
).group(1)
search = f"namespace {expected_name}"
if search in content:
# Check for both old style and C++17 nested namespace syntax
search_old = f"namespace {expected_name}"
search_new = f"namespace esphome::{expected_name}"
if search_old in content or search_new in content:
return None
return (
"Invalid namespace found in C++ file. All integration C++ files should put all "
"functions in a separate namespace that matches the integration's name. "
f"Please make sure the file contains {highlight(search)}"
f"Please make sure the file contains {highlight(search_old)} or {highlight(search_new)}"
)