Skip to content

chore(cli): Pass Celest's VM service URI to auxiliary start commands#270

Merged
dnys1 merged 1 commit intomainfrom
03-20-chore_cli_pass_celest_s_vm_service_uri_to_auxiliary_start_commands
Mar 21, 2025
Merged

chore(cli): Pass Celest's VM service URI to auxiliary start commands#270
dnys1 merged 1 commit intomainfrom
03-20-chore_cli_pass_celest_s_vm_service_uri_to_auxiliary_start_commands

Conversation

@dnys1
Copy link
Member

@dnys1 dnys1 commented Mar 21, 2025

When running celest start with an auxiliary command, pass the running server's VM service URI to the command as a Dart define.

This is useful when the auxiliary command is dart test or flutter test and allows package:celest_test to get live updates from the server in an integration test.

Copy link
Member Author

dnys1 commented Mar 21, 2025

Merge activity

  • Mar 20, 7:16 PM PDT: A user started a stack merge that includes this pull request via Graphite.
  • Mar 20, 7:21 PM PDT: Graphite rebased this pull request as part of a merge.
  • Mar 20, 7:23 PM PDT: A user merged this pull request with Graphite.

@dnys1 dnys1 changed the base branch from 03-20-chore_cli_add_full_reset_to_generated_client to graphite-base/270 March 21, 2025 02:18
@dnys1 dnys1 changed the base branch from graphite-base/270 to main March 21, 2025 02:20
When running `celest start` with an auxiliary command, pass the running server's VM service URI to the command as a Dart define.

This is useful when the auxiliary command is `dart test` or `flutter test` and allows `package:celest_test` to get live updates from the server in an integration test.
@dnys1 dnys1 force-pushed the 03-20-chore_cli_pass_celest_s_vm_service_uri_to_auxiliary_start_commands branch from b464163 to 9953331 Compare March 21, 2025 02:21
@dnys1 dnys1 merged commit 92cdbf2 into main Mar 21, 2025
4 checks passed
@dnys1 dnys1 deleted the 03-20-chore_cli_pass_celest_s_vm_service_uri_to_auxiliary_start_commands branch March 21, 2025 02:23
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.

1 participant