Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Convert True to true in query strings #156

Merged
merged 2 commits into from
Jun 8, 2023
Merged

Convert True to true in query strings #156

merged 2 commits into from
Jun 8, 2023

Conversation

nfx
Copy link
Contributor

@nfx nfx commented Jun 8, 2023

Fix #142

@codecov-commenter
Copy link

Codecov Report

Patch coverage: 40.00% and project coverage change: -0.01 ⚠️

Comparison is base (087cf3f) 53.15% compared to head (81bbc68) 53.15%.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #156      +/-   ##
==========================================
- Coverage   53.15%   53.15%   -0.01%     
==========================================
  Files          30       30              
  Lines       17980    17985       +5     
==========================================
+ Hits         9558     9560       +2     
- Misses       8422     8425       +3
Impacted Files Coverage Δ
databricks/sdk/core.py 67.13% <40.00%> (-0.20%) ⬇️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@nfx nfx merged commit 1046ddb into main Jun 8, 2023
@nfx nfx deleted the fix/142 branch June 8, 2023 12:48
@nfx nfx mentioned this pull request Jun 9, 2023
Merged
nfx added a commit that referenced this pull request Jun 9, 2023
@nfx
Release v0.1.9 (#160)
8660cff 
# Version changelog

## 0.1.9

* Added new services from OpenAPI spec
([#145](#145),
[#159](#159)).
* Added consistent usage of the `upload(path, IO)` and `download(path)
-> IO` across file-related operations
([#148](#148)).
* Added Databricks Metadata Service credential provider
([#139](#139),
[#130](#130)).
* Added exposing runtime credential provider without changing user
namespace
([#140](#140)).
* Added a check for `is not None` for primitive fields in `as_dict()`
([#147](#147)).
* Fixed bug related to boolean flags and convert `True` to `true` in
query strings
([#156](#156)).
* Fixed generation of external entities
([#146](#146)).
* Make u2m authentication work with new CLI
([#150](#150)).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

metastore.delete force does not force
2 participants
@nfx @codecov-commenter