Improve CLI parity and harden file handling#17
Merged
Conversation
Contributor
Author
|
Sorry for the first update saying I could not run the Dart checks. I was on my VPS environment then, and Dart was not installed there. I installed the SDK locally, ran the checks, fixed the analyzer feedback, and pushed the follow-up commit. Checks now passing:
|
Gorniaky
requested changes
Jul 4, 2026
Gorniaky
requested changes
Jul 7, 2026
Gorniaky
requested changes
Jul 7, 2026
Added validation to ensure 'apt' options are not empty.
Removed redundant success spinner call when team list is not empty.
Refactor backup command to require 'app' option and streamline directory handling.
Refactor backup handling methods to remove 'dir' parameter.
Refactor download function to improve error handling and response management.
Gorniaky
approved these changes
Jul 7, 2026
Contributor
Author
|
Thanks for merging my PR. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This brings the Dart CLI closer to the current Node CLI behavior and tightens a few places that were too easy to break in normal use.
What changed:
init,domain,subdomain,logs,backup,locale,ram,upload, and team/app variants were brought closer to the current command behavior.Validation:
dart format lib modules testdart analyzedart testgit diff --checkSorry about the earlier note saying I could not run Dart. I was working from my VPS environment at first and the Dart SDK was not available there. I reran the checks locally after installing the SDK and pushed the analyzer fixes in a follow-up commit.