Skip to content

fix(cli): allow calmhub URI to be set at command line#1836

Merged
markscott-ms merged 3 commits intofinos:mainfrom
willosborne:calm-hub-uri
Nov 21, 2025
Merged

fix(cli): allow calmhub URI to be set at command line#1836
markscott-ms merged 3 commits intofinos:mainfrom
willosborne:calm-hub-uri

Conversation

@willosborne
Copy link
Member

@willosborne willosborne commented Nov 21, 2025

Description

Adds missing CLI args, part of ongoing work towards #990

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📚 Documentation update
  • 🎨 Code style/formatting changes
  • ♻️ Refactoring (no functional changes)
  • ⚡ Performance improvements
  • ✅ Test additions or updates
  • 🔧 Chore (maintenance, dependencies, CI, etc.)

Affected Components

  • CLI (cli/)
  • Shared (shared/)
  • CALM Widgets (calm-widgets/)
  • CALM Hub (calm-hub/)
  • CALM Hub UI (calm-hub-ui/)
  • Documentation (docs/)
  • VS Code Extension (calm-plugins/vscode/)
  • Dependencies
  • CI/CD

Commit Message Format ✅

Testing

  • I have tested my changes locally
  • I have added/updated unit tests
  • All existing tests pass
image

Checklist

  • My commits follow the conventional commit format
  • I have updated documentation if necessary
  • I have added tests for my changes (if applicable)
  • My changes follow the project's coding standards

Copy link
Contributor

@markscott-ms markscott-ms left a comment

Choose a reason for hiding this comment

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

Whilst different to #1794 this looks fine. I will adjust #1794 if it remains as a PR

@markscott-ms markscott-ms merged commit 2fb373e into finos:main Nov 21, 2025
13 checks passed
@willosborne
Copy link
Member Author

@markscott-ms yes, this is just a missing CLI arg that I spotted. nothing to do with auth

@willosborne willosborne deleted the calm-hub-uri branch November 24, 2025 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants