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

Port auth tests from the JS SDK #313

Merged
merged 1 commit into from
Feb 24, 2023
Merged

Port auth tests from the JS SDK #313

merged 1 commit into from
Feb 24, 2023

Conversation

fjakobs
Copy link
Contributor

@fjakobs fjakobs commented Feb 23, 2023

@codecov-commenter
Copy link

Codecov Report

Base: 41.17% // Head: 41.17% // No change to project coverage 👍

Coverage data is based on head (965b1ad) compared to base (f7a75da).
Patch has no changes to coverable lines.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main     #313   +/-   ##
=======================================
  Coverage   41.17%   41.17%           
=======================================
  Files          46       46           
  Lines        2674     2674           
=======================================
  Hits         1101     1101           
  Misses       1485     1485           
  Partials       88       88

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

@nfx nfx merged commit 8696420 into main Feb 24, 2023
@nfx nfx deleted the fabian branch February 24, 2023 14:44
@nfx nfx mentioned this pull request Feb 24, 2023
Merged
nfx added a commit that referenced this pull request Feb 24, 2023
@nfx
Release v0.3.3 (#315)
79a30f7 
* Allow AAD SPN authentication on Databricks Account level
([#311](#311)).
* Port auth tests from the JS SDK
([#313](#313)).
* Skip loading `~/.databrickscfg` when not required
([#314](#314)).

Dependency updates:

* Bump golang.org/x/net from 0.6.0 to 0.7.0
([#312](#312)).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nfx nfx nfx approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fjakobs @codecov-commenter @nfx