Skip to content

Conversation

@heiruwu
Copy link
Contributor

@heiruwu heiruwu commented Sep 14, 2023

Because

  • allow users to authenticate with api-token for instill cloud

This commit

  • add support for api-token header

@linear
Copy link

linear bot commented Sep 14, 2023

@codecov
Copy link

codecov bot commented Sep 14, 2023

Codecov Report

Patch coverage: 65.51% and project coverage change: -0.20% ⚠️

Comparison is base (ccdbc81) 34.62% compared to head (939605a) 34.43%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #6      +/-   ##
==========================================
- Coverage   34.62%   34.43%   -0.20%     
==========================================
  Files          36       36              
  Lines        2917     2939      +22     
  Branches       39       43       +4     
==========================================
+ Hits         1010     1012       +2     
- Misses       1883     1899      +16     
- Partials       24       28       +4     
Flag Coverage Δ
unittests 34.43% <65.51%> (-0.20%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
instill_sdk/clients/connector.py 64.04% <61.53%> (-3.82%) ⬇️
instill_sdk/clients/model.py 46.77% <61.53%> (-1.97%) ⬇️
instill_sdk/clients/pipeline.py 66.66% <61.53%> (-4.57%) ⬇️
instill_sdk/clients/mgmt.py 74.64% <64.28%> (-5.66%) ⬇️
instill_sdk/clients/client.py 82.35% <100.00%> (+1.10%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@heiruwu heiruwu force-pushed the heiru/ins-1713-support-api-token-for-instill-cloud-authentication branch from 2fc110e to dadf160 Compare September 14, 2023 18:45
@heiruwu heiruwu force-pushed the main branch 3 times, most recently from af0f5b8 to ccdbc81 Compare September 14, 2023 19:05
@heiruwu heiruwu force-pushed the heiru/ins-1713-support-api-token-for-instill-cloud-authentication branch from dadf160 to 913f1a5 Compare September 14, 2023 19:10
@heiruwu heiruwu merged commit e041510 into main Sep 14, 2023
@heiruwu heiruwu deleted the heiru/ins-1713-support-api-token-for-instill-cloud-authentication branch September 14, 2023 19:20
heiruwu pushed a commit that referenced this pull request Oct 1, 2023
🤖 I have created a release *beep* *boop*
---


## 0.1.0 (2023-10-01)


### Features

* **auth:** adopt `Instill Core` auth
([#16](#16))
([86c0e85](86c0e85))
* **auth:** support api-token for instill cloud authentication
([#6](#6))
([e041510](e041510))
* **component,recipe:** support better component/recipe creation
([#22](#22))
([857e260](857e260))
* **config:** adopt pydantic for yaml config validation
([#9](#9))
([8903975](8903975))


### Bug Fixes

* **clients:** fix metadata overwrite
([#26](#26))
([e332cb0](e332cb0))


### Documentation

* **contributing,readme:** update docs and workflow for contributing
([#12](#12))
([6fb0c84](6fb0c84))
* **readme,notebook:** add a notebook example
([9f8d728](9f8d728))
* **readme:** add usage in README
([#24](#24))
([5af5f42](5af5f42))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants