)]}'
{"doc/source/contributor/code-conventions-and-testing.rst":[{"robot_id":"zuul","robot_run_id":"cf70a37111fc493a85cffc705de81d41","url":"https://zuul.teim.app/t/main/buildset/cf70a37111fc493a85cffc705de81d41","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"579fa21c6adf0eae25ae0f19af50b3a889ce557e","patch_set":1,"id":"7bb66b93_3d4a482e","line":428,"updated":"2026-02-28 12:35:33.000000000","message":"Documentation example for config.py shows \u0027from __future__ import annotations\u0027 but actual file doesn\u0027t have it\n\n**Severity**: SUGGESTION | **Confidence**: 0.7\n\n**Benefit**: The code example in the Centralised Configuration section shows an import that doesn\u0027t exist in the actual implementation, which could confuse contributors trying to match documentation to code.\n\n**Recommendation**:\nEither add \u0027from __future__ import annotations\u0027 to config.py, or update the documentation example to match the actual file content (without that import). For maximum accuracy, examples should exactly match the implementation they document.","commit_id":"a95d0c122d64785ed019ac2a85e16d7a0d14eed4"}],"watcher_dashboard/config.py":[{"robot_id":"zuul","robot_run_id":"cf70a37111fc493a85cffc705de81d41","url":"https://zuul.teim.app/t/main/buildset/cf70a37111fc493a85cffc705de81d41","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"579fa21c6adf0eae25ae0f19af50b3a889ce557e","patch_set":1,"id":"2a65dcb1_9a4e8704","line":13,"updated":"2026-02-28 12:35:33.000000000","message":"config.py lacks \u0027from __future__ import annotations\u0027 despite being documented as a mypy-enforced module\n\n**Severity**: SUGGESTION | **Confidence**: 0.8\n\n**Benefit**: The documentation in Type Hints section states \u0027Prefer ``from __future__ import annotations`` at the top of the file to enable forward references without runtime overhead.\u0027 and indicates config.py is type-hint enforced via mypy. Adding this import would align with the documented best practices.\n\n**Recommendation**:\nConsider adding \u0027from __future__ import annotations\u0027 as the first import after the license header and module docstring in config.py. This aligns with the documented best practices and is already present in api/watcher.py which is also mypy-enforced.","commit_id":"a95d0c122d64785ed019ac2a85e16d7a0d14eed4"},{"robot_id":"zuul","robot_run_id":"cf70a37111fc493a85cffc705de81d41","url":"https://zuul.teim.app/t/main/buildset/cf70a37111fc493a85cffc705de81d41","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"579fa21c6adf0eae25ae0f19af50b3a889ce557e","patch_set":1,"id":"d85e679e_826ae640","line":18,"updated":"2026-02-28 12:35:33.000000000","message":"config.py module docstring mentions functools.lru_cache but implementation uses functools.cache\n\n**Severity**: WARNING | **Confidence**: 0.8\n\n**Impact**: Documentation inconsistency - the docstring says \u0027cached with functools.lru_cache\u0027 but actual decorators use \u0027@functools.cache\u0027. While functools.cache is implemented as functools.lru_cache(maxsize\u003dNone), the reference should match the code for accuracy.\n\n**Suggestion**:\nUpdate docstring line 18 from \u0027functools.lru_cache\u0027 to \u0027functools.cache\u0027 for consistency with the actual implementation. The current docstring reads: \u0027Functions that return immutable values are cached with ``functools.lru_cache``\u0027 but should read \u0027``functools.cache``\u0027.","commit_id":"a95d0c122d64785ed019ac2a85e16d7a0d14eed4"}]}
