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.

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

feat: Make state client versioning the default, remove option #1938

Merged
merged 2 commits into from
Oct 18, 2024

Conversation

erezrokah
Copy link
Member

Summary

Fixes https://github.com/cloudquery/cloudquery-issues/issues/2435


Use the following steps to ensure your PR is ready to be reviewed

  • Read the contribution guidelines πŸ§‘β€πŸŽ“
  • Run go fmt to format your code πŸ–Š
  • Lint your changes via golangci-lint run 🚨 (install golangci-lint here)
  • Update or add tests πŸ§ͺ
  • Ensure the status checks below are successful βœ…

@erezrokah erezrokah requested a review from a team as a code owner October 17, 2024 11:40
@github-actions github-actions bot added the feat label Oct 17, 2024
@@ -21,9 +21,7 @@ type Client interface {
Close() error
}

type ClientOptions struct {
Versioned bool
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will break compilation for few plugins, I'll fix them once the SDK it released

Copy link
Contributor

@marianogappa marianogappa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't apply to Python plugins, which do support backend options, and I'm pretty certain versioning is not supported.

@kodiakhq kodiakhq bot merged commit f105651 into main Oct 18, 2024
8 checks passed
@kodiakhq kodiakhq bot deleted the feat/enable_state_client_versioning branch October 18, 2024 08:01
kodiakhq bot pushed a commit that referenced this pull request Oct 18, 2024
πŸ€– I have created a release *beep* *boop*
---


## [4.67.0](v4.66.1...v4.67.0) (2024-10-18)


### Features

* Make state client versioning the default, remove option ([#1938](#1938)) ([f105651](f105651))


### Bug Fixes

* **deps:** Update module github.com/cloudquery/plugin-pb-go to v1.23.0 ([#1934](#1934)) ([ea8b17a](ea8b17a))
* **deps:** Update module github.com/cloudquery/plugin-pb-go to v1.23.1 ([#1936](#1936)) ([0b152ba](0b152ba))
* **deps:** Update module github.com/cloudquery/plugin-pb-go to v1.24.0 ([#1937](#1937)) ([d9e6f47](d9e6f47))

---
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants