Add --dry-run flag to bernstein compose command#274
Conversation
Implements issue sipyourdrink-ltd#253 by adding a --dry-run flag that prints the JSON payload that would be sent without actually calling the API. This allows users to preview and validate task definitions before submitting them.
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (1 files) [Incremental review]
Reviewed by grok-code-fast-1 · 137,354 tokens |
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (1 files)
|
1 similar comment
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (1 files)
|
chernistry
left a comment
chernistry
left a comment
There was a problem hiding this comment.
Nice implementation, thanks! One small fix needed before merge:
The last line of task_cmd.py lost its trailing newline (see \ No newline at end of file in the diff). Please add it back — most editors do this automatically, just make sure the file ends with a newline character.
Once that's fixed, LGTM 👍
chernistry
left a comment
chernistry
left a comment
There was a problem hiding this comment.
Fixed the trailing newline myself — good work, thanks for the contribution! 👍
|
Add --dry-run flag to bernstein compose command
2 participants
This PR adds a --dry-run flag to the bernstein compose command that prints the JSON payload that would be sent to the server without actually making the API call. This allows users to preview and validate task definitions before submitting them.