Skip to content

Conversation

@erezrokah
Copy link
Member

Summary

Goes with cloudquery/plugin-pb-go#62. I'll update the PR once we release a new version of plugin-pb-go


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 July 14, 2023 08:44
@github-actions github-actions bot added the fix label Jul 14, 2023
@erezrokah erezrokah force-pushed the fix/add_no_connection branch from 20763c5 to 8bf79f7 Compare July 14, 2023 08:45
@github-actions
Copy link

github-actions bot commented Jul 14, 2023

⏱️ Benchmark results

  • Glob-8 ns/op: 100.2

@codecov
Copy link

codecov bot commented Jul 14, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (fb124a2) 47.17% compared to head (8bf79f7) 47.17%.

❗ Current head 8bf79f7 differs from pull request most recent head 0eb9788. Consider uploading reports for the commit 0eb9788 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1092   +/-   ##
=======================================
  Coverage   47.17%   47.17%           
=======================================
  Files          84       84           
  Lines        7728     7728           
=======================================
  Hits         3646     3646           
  Misses       3748     3748           
  Partials      334      334           
Impacted Files Coverage Δ
internal/servers/plugin/v3/plugin.go 35.67% <0.00%> (ø)

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

@kodiakhq kodiakhq bot merged commit ba16cfd into cloudquery:main Jul 14, 2023
kodiakhq bot pushed a commit that referenced this pull request Jul 14, 2023
🤖 I have created a release *beep* *boop*
---


## [4.1.1](v4.1.0...v4.1.1) (2023-07-14)


### Bug Fixes

* Add `NoConnection` to init request ([#1092](#1092)) ([ba16cfd](ba16cfd))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
kodiakhq bot pushed a commit to cloudquery/cloudquery that referenced this pull request Jul 14, 2023

#### Summary

Goes with cloudquery/plugin-sdk#1092 and cloudquery/plugin-pb-go#62.
This ensures the `cloudquery migrate` command does not require authentication to AWS, GCP, Azure, etc.

<!--
@erezrokah erezrokah deleted the fix/add_no_connection branch July 14, 2023 10:20
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