Skip to content

test: Add URL argument to command line#52

Merged
tomasz-blasz merged 1 commit intomainfrom
topic/add-url-arg-in-api-test
Mar 5, 2026
Merged

test: Add URL argument to command line#52
tomasz-blasz merged 1 commit intomainfrom
topic/add-url-arg-in-api-test

Conversation

@tomasz-blasz
Copy link
Contributor

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a --url command-line option to the API test app so developers can point the client at an arbitrary Firebolt WebSocket endpoint, and updates docs/changelog accordingly.

Changes:

  • Add --url argument parsing to test/api_test_app/main.cpp.
  • Update API test app usage documentation/help text and repository changelog.
  • Adjust argument parsing loop condition in build.sh.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
test/api_test_app/main.cpp Adds --url handling and expands --help output.
test/api_test_app/README.md Documents --url usage and endpoint selection priority.
build.sh Changes the CLI-arg parsing loop condition.
CHANGELOG.md Notes the new --url capability in the API test app.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tomasz-blasz tomasz-blasz force-pushed the topic/add-url-arg-in-api-test branch from cd7ad8b to ac2fcf9 Compare March 5, 2026 10:55
@tomasz-blasz tomasz-blasz merged commit 36d94f1 into main Mar 5, 2026
12 checks passed
@tomasz-blasz tomasz-blasz deleted the topic/add-url-arg-in-api-test branch March 5, 2026 11:00
@github-actions github-actions bot locked and limited conversation to collaborators Mar 5, 2026
@tomasz-blasz
Copy link
Contributor Author

🎉 This PR is included in version 0.5.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants