Skip to content

Conversation

@heiruwu
Copy link
Contributor

@heiruwu heiruwu commented Sep 19, 2023

Because

  • provide better UX for configuring SDK

This commit

  • unify config location for instill products
  • adopt yaml config file for target hosts configuration
  • adopt pydantic for config validation

@linear
Copy link

linear bot commented Sep 19, 2023

@codecov
Copy link

codecov bot commented Sep 19, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.29% ⚠️

Comparison is base (097a1c1) 33.93% compared to head (bc25c89) 33.65%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #9      +/-   ##
==========================================
- Coverage   33.93%   33.65%   -0.29%     
==========================================
  Files          36       37       +1     
  Lines        2994     3025      +31     
  Branches       42       45       +3     
==========================================
+ Hits         1016     1018       +2     
- Misses       1950     1979      +29     
  Partials       28       28              
Files Changed Coverage Δ
instill_sdk/configuration/__init__.py 0.00% <0.00%> (ø)

... and 1 file with indirect coverage changes

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

@heiruwu heiruwu merged commit 8903975 into main Sep 19, 2023
@heiruwu heiruwu deleted the heiru/ins-1959-adopt-config-handling-lib branch September 19, 2023 08:39
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