Skip to content

Comments

fix(plugin-tables): Add missing skip_dependent_tables#1136

Merged
erezrokah merged 1 commit intocloudquery:mainfrom
erezrokah:fix/pass_skip_dependant_tables
Aug 8, 2023
Merged

fix(plugin-tables): Add missing skip_dependent_tables#1136
erezrokah merged 1 commit intocloudquery:mainfrom
erezrokah:fix/pass_skip_dependant_tables

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