Skip to content

fix(cli): handle missing repo binding in register#63

Merged
TrevorBasinger merged 1 commit intomainfrom
tb/register-friendly-binding-error
Apr 17, 2026
Merged

fix(cli): handle missing repo binding in register#63
TrevorBasinger merged 1 commit intomainfrom
tb/register-friendly-binding-error

Conversation

@TrevorBasinger
Copy link
Copy Markdown
Member

Summary

  • handle the missing GLaaS repo binding path in roar register as a user-facing CLI error
  • preserve the helpful guidance to link the repo to a TReqs owner/project or rerun with --public
  • add regression coverage to ensure this path does not print a Python traceback

Testing

  • uv run --extra dev ruff format roar/publish_auth.py roar/application/publish/service.py
    tests/integration/test_public_publish_intent_cli.py
  • uv run --extra dev ruff check roar/publish_auth.py roar/application/publish/service.py
    tests/integration/test_public_publish_intent_cli.py
  • uv run --extra dev pytest tests/integration/test_public_publish_intent_cli.py -q
  • uv run --extra dev ruff check .
  • uv run --extra dev mypy roar
  • uv run --extra dev pytest -m "not live_glaas and not ebpf"

@TrevorBasinger TrevorBasinger marked this pull request as ready for review April 17, 2026 17:24
Copy link
Copy Markdown
Member

@christophergeyer christophergeyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢 it

@TrevorBasinger TrevorBasinger merged commit 53ff155 into main Apr 17, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants