Skip to content

Commit 12a7d06

Browse files
committed
Address review comments from Copilot
1 parent c737d50 commit 12a7d06

3 files changed

Lines changed: 8 additions & 3 deletions

File tree

src/fastapi_cloud_cli/commands/whoami.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ def whoami() -> Any:
2727

2828
print(f"⚡ [bold]{data['email']}[/bold]")
2929

30-
# Deplotment token status
30+
# Deployment token status
3131
if identity.deploy_token is not None:
3232
print(
3333
"⚡ [bold]Using API token from environment variable for "

src/fastapi_cloud_cli/utils/sentry.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,9 @@
77

88

99
def init_sentry() -> None:
10-
"""Initialize Sentry error tracking only if user is logged in."""
10+
"""
11+
Initialize Sentry error tracking only if user is logged in or has a deployment token.
12+
"""
1113
identity = Identity(
1214
prefer_auth_mode="token" # Use auth_mode="token" as it has a fallback to user token
1315
)

tests/test_cli_deploy.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1563,7 +1563,10 @@ def test_cancel_upload_swallows_exceptions(
15631563
)
15641564
@pytest.mark.respx
15651565
def test_deploy_successfully_with_token(
1566-
logged_out_cli: None, command: str, tmp_path: Path, respx_mock: respx.MockRouter
1566+
logged_out_cli: None,
1567+
command: list[str],
1568+
tmp_path: Path,
1569+
respx_mock: respx.MockRouter,
15671570
) -> None:
15681571
app_data = _get_random_app()
15691572
team_data = _get_random_team()

0 commit comments

Comments
 (0)