Skip to content

Add database start and database stop commands#144

Merged
kangasta merged 1 commit into
masterfrom
feat/database-start-stop
Jul 5, 2022
Merged

Add database start and database stop commands#144
kangasta merged 1 commit into
masterfrom
feat/database-start-stop

Conversation

@kangasta
Copy link
Copy Markdown
Member

@kangasta kangasta commented Jul 1, 2022

Add database connection list, database connection cancel, database start, and database stop commands.

@kangasta kangasta force-pushed the feat/database-start-stop branch 2 times, most recently from 8e2d11c to 430d75e Compare July 4, 2022 13:08
@kangasta kangasta marked this pull request as ready for review July 4, 2022 13:09
@kangasta kangasta force-pushed the feat/database-start-stop branch from 430d75e to 4793430 Compare July 4, 2022 16:20
Comment thread CHANGELOG.md
### Added
- Add `--show-ip-addresses` flag to `server list` command to optionally include IP addresses in command output.
- Add `database connection list`, and `database connection cancel` commands.
- Add `database connection list`, `database connection cancel`, `database start`, and `database stop` commands.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

personally I would keep each command as a separate item on the list, for me that easier to quickly parse with eyes. Not a blocker or anything ofc.

Copy link
Copy Markdown
Contributor

@ka-myl ka-myl left a comment

Choose a reason for hiding this comment

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

Code looks good. Tested locally and works as intended

@kangasta kangasta merged commit 8c81aca into master Jul 5, 2022
@kangasta kangasta deleted the feat/database-start-stop branch July 5, 2022 08:56
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