-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added OnlyPayload property for method #522
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #522 +/- ##
==========================================
- Coverage 18.76% 18.76% -0.01%
==========================================
Files 85 85
Lines 9187 9188 +1
==========================================
Hits 1724 1724
- Misses 7315 7316 +1
Partials 148 148
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does this mean? Not familiar with the property.
@pietern renamed to |
@@ -154,6 +154,10 @@ func (m *Method) IsCrudCreate() bool { | |||
return m.operation.Crud == "create" | |||
} | |||
|
|||
func (m *Method) JsonOnly() bool { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
func (m *Method) JsonOnly() bool { | |
func (m *Method) IsJsonOnly() bool { |
## Changes Fixed jobs create command to only accept JSON payload. Note: relies on this PR from Go SDK databricks/databricks-sdk-go#522 ## Tests ``` andrew.nester@HFW9Y94129 cli % ./cli jobs create -h Create a new job. Create a new job. Usage: databricks jobs create [flags] Flags: -h, --help help for create --json JSON either inline JSON string or @path/to/file.json with request body (default JSON (0 bytes)) Global Flags: -e, --environment string bundle environment to use (if applicable) --log-file file file to write logs to (default stderr) --log-format type log output format (text or json) (default text) --log-level format log level (default disabled) -o, --output type output type: text or json (default text) -p, --profile string ~/.databrickscfg profile --progress-format format format for progress logs (append, inplace, json) (default default) ```
Changes
Added OnlyPayload property for method
Tests
make test
passingmake fmt
appliedrelevant integration tests applied