)]}'
{"oslo_config/generator.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"c470bbc7ad98962c0253aff598c687b71f094a1e","unresolved":true,"context_lines":[{"line_number":197,"context_line":""},{"line_number":198,"context_line":""},{"line_number":199,"context_line":"def _format_type_name("},{"line_number":200,"context_line":"    opt_type: types.ConfigType[Any] | type[str | int | float],"},{"line_number":201,"context_line":") -\u003e str:"},{"line_number":202,"context_line":"    \"\"\"Format the type name to use in describing an option\"\"\""},{"line_number":203,"context_line":"    if isinstance(opt_type, types.ConfigType):"}],"source_content_type":"text/x-python","patch_set":2,"id":"334307ba_b7153d89","line":200,"range":{"start_line":200,"start_character":14,"end_line":200,"end_character":62},"updated":"2026-10-13 15:33:48.000000000","message":"I wonder if this conflicts with the fallback in L211 ?","commit_id":"a26e782865651a41599bbd80bf5d9809cf907d87"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6337c8e7e042b12bef3e985f572566b1d6b6977c","unresolved":false,"context_lines":[{"line_number":197,"context_line":""},{"line_number":198,"context_line":""},{"line_number":199,"context_line":"def _format_type_name("},{"line_number":200,"context_line":"    opt_type: types.ConfigType[Any] | type[str | int | float],"},{"line_number":201,"context_line":") -\u003e str:"},{"line_number":202,"context_line":"    \"\"\"Format the type name to use in describing an option\"\"\""},{"line_number":203,"context_line":"    if isinstance(opt_type, types.ConfigType):"}],"source_content_type":"text/x-python","patch_set":2,"id":"a55240b7_349ea5b1","line":200,"range":{"start_line":200,"start_character":14,"end_line":200,"end_character":62},"in_reply_to":"334307ba_b7153d89","updated":"2026-10-13 15:39:52.000000000","message":"It does, but we can use types to indicate what we expect while still providing a runtime fallback for other types. In a typed code base, we should never get other types beyond these in here.","commit_id":"a26e782865651a41599bbd80bf5d9809cf907d87"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"ddc125db8429c14d421adcbb0c3d97f34eb2c2ae","unresolved":false,"context_lines":[{"line_number":197,"context_line":""},{"line_number":198,"context_line":""},{"line_number":199,"context_line":"def _format_type_name("},{"line_number":200,"context_line":"    opt_type: types.ConfigType[Any] | type[str | int | float],"},{"line_number":201,"context_line":") -\u003e str:"},{"line_number":202,"context_line":"    \"\"\"Format the type name to use in describing an option\"\"\""},{"line_number":203,"context_line":"    if isinstance(opt_type, types.ConfigType):"}],"source_content_type":"text/x-python","patch_set":2,"id":"05f285eb_6046053b","line":200,"range":{"start_line":200,"start_character":14,"end_line":200,"end_character":62},"in_reply_to":"a55240b7_349ea5b1","updated":"2026-10-13 15:41:33.000000000","message":"OK. That makes sense. Thanks for clarification.","commit_id":"a26e782865651a41599bbd80bf5d9809cf907d87"}],"oslo_config/validator.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"c470bbc7ad98962c0253aff598c687b71f094a1e","unresolved":true,"context_lines":[{"line_number":118,"context_line":") -\u003e bool:"},{"line_number":119,"context_line":"    \"\"\"Compares the current and sample configuration and reports differences"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"    :param section: ConfigParser instance"},{"line_number":122,"context_line":"    :param opt_data: machine readable data from the generator instance"},{"line_number":123,"context_line":"    :param conf: ConfigOpts instance"},{"line_number":124,"context_line":"    :returns: boolean wether or not warnings were reported"}],"source_content_type":"text/x-python","patch_set":2,"id":"e846a8de_e42d8315","line":121,"range":{"start_line":121,"start_character":11,"end_line":121,"end_character":18},"updated":"2026-10-13 15:33:48.000000000","message":"(nit) sections","commit_id":"a26e782865651a41599bbd80bf5d9809cf907d87"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6337c8e7e042b12bef3e985f572566b1d6b6977c","unresolved":false,"context_lines":[{"line_number":118,"context_line":") -\u003e bool:"},{"line_number":119,"context_line":"    \"\"\"Compares the current and sample configuration and reports differences"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"    :param section: ConfigParser instance"},{"line_number":122,"context_line":"    :param opt_data: machine readable data from the generator instance"},{"line_number":123,"context_line":"    :param conf: ConfigOpts instance"},{"line_number":124,"context_line":"    :returns: boolean wether or not warnings were reported"}],"source_content_type":"text/x-python","patch_set":2,"id":"23f06ca5_367b2651","line":121,"range":{"start_line":121,"start_character":11,"end_line":121,"end_character":18},"in_reply_to":"e846a8de_e42d8315","updated":"2026-10-13 15:39:52.000000000","message":"Done","commit_id":"a26e782865651a41599bbd80bf5d9809cf907d87"}]}
