Skip to content

Commit 9de04bc

Browse files
committed
fix: support python 3.11 f-string parsing
1 parent ac36708 commit 9de04bc

1 file changed

Lines changed: 18 additions & 9 deletions

File tree

src/brigade/work_cmd.py

Lines changed: 18 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -5563,7 +5563,8 @@ def brief(*, target: Path, limit: int = 3, json_output: bool = False) -> int:
55635563

55645564
inbox_hygiene = payload.get("inbox_hygiene") if isinstance(payload.get("inbox_hygiene"), dict) else {}
55655565
if inbox_hygiene:
5566-
print(f"inbox_hygiene: {'ok' if inbox_hygiene.get('issue_count') == 0 else f'{inbox_hygiene.get('issue_count')} issue(s)'}")
5566+
issue_count = inbox_hygiene.get("issue_count")
5567+
print(f"inbox_hygiene: {'ok' if issue_count == 0 else f'{issue_count} issue(s)'}")
55675568
top_inbox = inbox_hygiene.get("top_issue") if isinstance(inbox_hygiene.get("top_issue"), dict) else None
55685569
if top_inbox:
55695570
print(
@@ -5621,7 +5622,8 @@ def brief(*, target: Path, limit: int = 3, json_output: bool = False) -> int:
56215622
memory_care = payload.get("memory_care") if isinstance(payload.get("memory_care"), dict) else {}
56225623
if memory_care:
56235624
print(f"memory_care_config: {memory_care.get('config_path')}")
5624-
print(f"memory_care_health: {'ok' if memory_care.get('issue_count') == 0 else f'{memory_care.get('issue_count')} issue(s)'}")
5625+
issue_count = memory_care.get("issue_count")
5626+
print(f"memory_care_health: {'ok' if issue_count == 0 else f'{issue_count} issue(s)'}")
56255627
top_memory = memory_care.get("top_issue") if isinstance(memory_care.get("top_issue"), dict) else None
56265628
if top_memory:
56275629
print(
@@ -5633,7 +5635,8 @@ def brief(*, target: Path, limit: int = 3, json_output: bool = False) -> int:
56335635
security_health = payload.get("security_health") if isinstance(payload.get("security_health"), dict) else {}
56345636
if security_health:
56355637
print(f"security_config: {security_health.get('config_path')}")
5636-
print(f"security_health: {'ok' if security_health.get('issue_count') == 0 else f'{security_health.get('issue_count')} issue(s)'}")
5638+
issue_count = security_health.get("issue_count")
5639+
print(f"security_health: {'ok' if issue_count == 0 else f'{issue_count} issue(s)'}")
56375640
top_security = security_health.get("top_finding") if isinstance(security_health.get("top_finding"), dict) else None
56385641
if top_security:
56395642
print(
@@ -5646,7 +5649,8 @@ def brief(*, target: Path, limit: int = 3, json_output: bool = False) -> int:
56465649
backup_health = payload.get("backup_health") if isinstance(payload.get("backup_health"), dict) else {}
56475650
if backup_health:
56485651
print(f"backup_config: {backup_health.get('config_path')}")
5649-
print(f"backup_health: {'ok' if backup_health.get('issue_count') == 0 else f'{backup_health.get('issue_count')} issue(s)'}")
5652+
issue_count = backup_health.get("issue_count")
5653+
print(f"backup_health: {'ok' if issue_count == 0 else f'{issue_count} issue(s)'}")
56505654
top_backup = backup_health.get("top_issue") if isinstance(backup_health.get("top_issue"), dict) else None
56515655
if top_backup:
56525656
print(
@@ -5658,7 +5662,8 @@ def brief(*, target: Path, limit: int = 3, json_output: bool = False) -> int:
56585662
tool_catalog = payload.get("tool_catalog") if isinstance(payload.get("tool_catalog"), dict) else {}
56595663
if tool_catalog:
56605664
print(f"tool_config: {tool_catalog.get('config_path')}")
5661-
print(f"tool_catalog: {'ok' if tool_catalog.get('issue_count') == 0 else f'{tool_catalog.get('issue_count')} issue(s)'}")
5665+
issue_count = tool_catalog.get("issue_count")
5666+
print(f"tool_catalog: {'ok' if issue_count == 0 else f'{issue_count} issue(s)'}")
56625667
top_tool = tool_catalog.get("top_issue") if isinstance(tool_catalog.get("top_issue"), dict) else None
56635668
if top_tool:
56645669
print(
@@ -5699,9 +5704,10 @@ def brief(*, target: Path, limit: int = 3, json_output: bool = False) -> int:
56995704

57005705
roadmap_completion = payload.get("roadmap_completion") if isinstance(payload.get("roadmap_completion"), dict) else {}
57015706
if roadmap_completion:
5707+
issue_count = roadmap_completion.get("issue_count")
57025708
print(
57035709
"roadmap_completion: "
5704-
f"{'ok' if roadmap_completion.get('issue_count') == 0 else f'{roadmap_completion.get('issue_count')} issue(s)'}"
5710+
f"{'ok' if issue_count == 0 else f'{issue_count} issue(s)'}"
57055711
)
57065712
top_roadmap = roadmap_completion.get("top_issue") if isinstance(roadmap_completion.get("top_issue"), dict) else None
57075713
if top_roadmap:
@@ -5710,7 +5716,8 @@ def brief(*, target: Path, limit: int = 3, json_output: bool = False) -> int:
57105716
repo_fleet = payload.get("repo_fleet") if isinstance(payload.get("repo_fleet"), dict) else {}
57115717
if repo_fleet:
57125718
print(f"repo_fleet_config: {repo_fleet.get('config_path')}")
5713-
print(f"repo_fleet: {'ok' if repo_fleet.get('issue_count') == 0 else f'{repo_fleet.get('issue_count')} issue(s)'}")
5719+
issue_count = repo_fleet.get("issue_count")
5720+
print(f"repo_fleet: {'ok' if issue_count == 0 else f'{issue_count} issue(s)'}")
57145721
top_repo = repo_fleet.get("top_issue") if isinstance(repo_fleet.get("top_issue"), dict) else None
57155722
if top_repo:
57165723
print(f"repo_fleet_top_issue: {top_repo.get('name')} {_short(str(top_repo.get('detail', '')))}")
@@ -5725,7 +5732,8 @@ def brief(*, target: Path, limit: int = 3, json_output: bool = False) -> int:
57255732

57265733
project_consolidation = payload.get("project_consolidation") if isinstance(payload.get("project_consolidation"), dict) else {}
57275734
if project_consolidation:
5728-
print(f"project_consolidation: {'ok' if project_consolidation.get('issue_count') == 0 else f'{project_consolidation.get('issue_count')} issue(s)'}")
5735+
issue_count = project_consolidation.get("issue_count")
5736+
print(f"project_consolidation: {'ok' if issue_count == 0 else f'{issue_count} issue(s)'}")
57295737
top_project = project_consolidation.get("top_issue") if isinstance(project_consolidation.get("top_issue"), dict) else None
57305738
if top_project:
57315739
print(f"project_consolidation_top_issue: {top_project.get('name')} {_short(str(top_project.get('detail', '')))}")
@@ -5742,7 +5750,8 @@ def brief(*, target: Path, limit: int = 3, json_output: bool = False) -> int:
57425750
latest_report = operator_report.get("latest") if isinstance(operator_report.get("latest"), dict) else None
57435751
if latest_report:
57445752
print(f"operator_report_latest: {latest_report.get('report_id')} {latest_report.get('created_at')}")
5745-
print(f"operator_report: {'ok' if operator_report.get('issue_count') == 0 else f'{operator_report.get('issue_count')} issue(s)'}")
5753+
issue_count = operator_report.get("issue_count")
5754+
print(f"operator_report: {'ok' if issue_count == 0 else f'{issue_count} issue(s)'}")
57465755
top_report = operator_report.get("top_issue") if isinstance(operator_report.get("top_issue"), dict) else None
57475756
if top_report:
57485757
print(f"operator_report_top_issue: {top_report.get('name')} {_short(str(top_report.get('detail', '')))}")

0 commit comments

Comments
 (0)