Skip to content

Conversation

@hermanschaaf
Copy link
Member

@hermanschaaf hermanschaaf commented Jun 30, 2023

#1026 should be merged first

@github-actions github-actions bot added feat and removed feat labels Jun 30, 2023
@codecov
Copy link

codecov bot commented Jun 30, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.24 ⚠️

Comparison is base (abe2557) 48.03% compared to head (6550fdb) 47.79%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1027      +/-   ##
==========================================
- Coverage   48.03%   47.79%   -0.24%     
==========================================
  Files          77       77              
  Lines        7455     7492      +37     
==========================================
  Hits         3581     3581              
- Misses       3543     3580      +37     
  Partials      331      331              
Impacted Files Coverage Δ
internal/clients/state/v3/state.go 0.00% <0.00%> (ø)
internal/servers/plugin/v3/plugin.go 37.85% <0.00%> (-8.04%) ⬇️
plugin/plugin.go 37.50% <0.00%> (ø)
plugin/plugin_destination.go 33.33% <0.00%> (-26.67%) ⬇️

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

Copy link
Contributor

@yevgenypats yevgenypats left a comment

Choose a reason for hiding this comment

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

LGTM. I think go.mod conflict and lint fixes are needed.

@github-actions
Copy link

⏱️ Benchmark results

Comparing with abe2557

  • Glob-2 ns/op: 257.5 ⬆️ 14.91% increase vs. abe2557

@kodiakhq kodiakhq bot merged commit 09fb4ce into main Jun 30, 2023
@kodiakhq kodiakhq bot deleted the implement-read branch June 30, 2023 10:58
kodiakhq bot pushed a commit that referenced this pull request Jun 30, 2023
🤖 I have created a release *beep* *boop*
---


## [4.4.0-rc1](v4.3.1-rc1...v4.4.0-rc1) (2023-06-30)


### Features

* Implement plugin Read ([#1027](#1027)) ([09fb4ce](09fb4ce))


### Bug Fixes

* **deps:** Update module github.com/cloudquery/plugin-pb-go to v1.4.0 ([#1005](#1005)) ([40f1c77](40f1c77))
* Update to plugin-pb v1.5.0 ([#1026](#1026)) ([abe2557](abe2557))

---
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.

3 participants