Skip to content

Conversation

@erezrokah
Copy link
Member

Summary

Goes with cloudquery/plugin-pb#17 and cloudquery/plugin-pb-go#84


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 yevgenypats as a code owner August 8, 2023 14:07
@github-actions github-actions bot added the fix label Aug 8, 2023
@erezrokah erezrokah added automerge and removed fix labels Aug 8, 2023
@github-actions
Copy link

github-actions bot commented Aug 8, 2023

⏱️ Benchmark results

Comparing with ed96059

  • Glob-8 ns/op: 99.39 ⬆️ 0.35% increase vs. ed96059

@codecov
Copy link

codecov bot commented Aug 8, 2023

Codecov Report

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

Comparison is base (ed96059) 48.61% compared to head (49a8fa4) 48.60%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1136      +/-   ##
==========================================
- Coverage   48.61%   48.60%   -0.01%     
==========================================
  Files          85       85              
  Lines        7819     7820       +1     
==========================================
  Hits         3801     3801              
- Misses       3684     3685       +1     
  Partials      334      334              
Files Changed Coverage Ξ”
internal/servers/plugin/v3/plugin.go 34.71% <0.00%> (-0.19%) ⬇️

β˜” View full report in Codecov by Sentry.
πŸ“’ Have feedback on the report? Share it here.

@erezrokah
Copy link
Member Author

Going to merge this cc @yevgenypats

@erezrokah erezrokah merged commit 65e9f1a into cloudquery:main Aug 8, 2023
@erezrokah erezrokah deleted the fix/pass_skip_dependant_tables branch August 8, 2023 14:45
kodiakhq bot pushed a commit that referenced this pull request Aug 8, 2023
πŸ€– I have created a release *beep* *boop*
---


## [4.3.1](v4.3.0...v4.3.1) (2023-08-08)


### Bug Fixes

* **plugin-tables:** Add missing `skip_dependent_tables` ([#1136](#1136)) ([65e9f1a](65e9f1a))

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