Skip to content

refactor(cli): Abstract some components#365

Merged
dnys1 merged 1 commit intomainfrom
04-27-refactor_cli_abstract_some_components
Apr 28, 2025
Merged

refactor(cli): Abstract some components#365
dnys1 merged 1 commit intomainfrom
04-27-refactor_cli_abstract_some_components

Conversation

@dnys1
Copy link
Member

@dnys1 dnys1 commented Apr 28, 2025

  • Lift up stop signal so that all commands have access to functionality
  • Move cloud-related logic to a unified CloudRepository

Copy link
Member Author

dnys1 commented Apr 28, 2025

@dnys1 dnys1 force-pushed the 04-27-refactor_cli_abstract_some_components branch from 1302c96 to e5322ef Compare April 28, 2025 01:54
@dnys1 dnys1 force-pushed the 04-27-chore_celest_add_logstatements_parameter_to_connect_ branch from 0da498d to bad4183 Compare April 28, 2025 01:54
@dnys1 dnys1 force-pushed the 04-27-refactor_cli_abstract_some_components branch from e5322ef to ff3842e Compare April 28, 2025 02:05
Copy link
Member Author

dnys1 commented Apr 28, 2025

Merge activity

  • Apr 27, 7:30 PM PDT: A user started a stack merge that includes this pull request via Graphite.
  • Apr 27, 7:34 PM PDT: Graphite rebased this pull request as part of a merge.
  • Apr 27, 7:35 PM PDT: @dnys1 merged this pull request with Graphite.

@dnys1 dnys1 changed the base branch from 04-27-chore_celest_add_logstatements_parameter_to_connect_ to graphite-base/365 April 28, 2025 02:31
@dnys1 dnys1 changed the base branch from graphite-base/365 to main April 28, 2025 02:33
- Lift up stop signal so that all commands have access to functionality
- Move cloud-related logic to a unified `CloudRepository`
@dnys1 dnys1 force-pushed the 04-27-refactor_cli_abstract_some_components branch from ff3842e to a2781f0 Compare April 28, 2025 02:33
@dnys1 dnys1 merged commit e2337c3 into main Apr 28, 2025
6 checks passed
@dnys1 dnys1 deleted the 04-27-refactor_cli_abstract_some_components branch April 28, 2025 02:35
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