Skip to content

feat!: CloudQuery V2.#1463

Merged
yevgenypats merged 47 commits intomainfrom
feat/cloudquery-cli-v2
Sep 6, 2022
Merged

feat!: CloudQuery V2.#1463
yevgenypats merged 47 commits intomainfrom
feat/cloudquery-cli-v2

Conversation

@yevgenypats
Copy link
Copy Markdown
Contributor

This is a rebased version of #1201
goes together with cloudquery/plugin-sdk#4

This is V2 of CloudQuery CLI together with the new https://github.com/cloudquery/plugin-sdk

  • Support for multiple databases. plugins are sending info back to CLI

  • seperate CLI configuration from plugins configuration.

  • Moved to plain gRPC (no more go-plugin)

  • zerolog everwhere

  • no custom UI implementations

  • no bloated/unneeded abstraction layers

Issues that should be fixed:

#1104
#1055
#983
#859
#858
#888
#857
#854
#904
#751
#539
#392
#292
#159
#908
#840

Summary


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

  • Read the contribution guidelines 🧑‍🎓
  • Test locally on your own infrastructure
  • 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 ✅

@yevgenypats yevgenypats marked this pull request as draft August 17, 2022 11:53
@yevgenypats yevgenypats mentioned this pull request Aug 17, 2022
6 tasks
@github-actions github-actions bot added breaking and removed feat labels Aug 17, 2022
yevgenypats and others added 6 commits September 6, 2022 12:30
test: Add more tests to cli
* feat: Add example of list/detail codegen

* make naming more reasonable

* finalizing gcp

* finalize gcp migration
* update to latest sdk

* feat: Move to new SDK

* more fixes to autogeneration

* more fixes to gcp

* feat: Add sentry

* remove old faker
@yevgenypats yevgenypats force-pushed the feat/cloudquery-cli-v2 branch from 56f2203 to 7f3548d Compare September 6, 2022 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants