Skip to content

feat: Modify Heroku plugin for v2#1642

Closed
hermanschaaf wants to merge 55 commits intocloudquery:feat/cloudquery-cli-v2from
hermanschaaf:feat/cloudquery-cli-v2-heroku
Closed

feat: Modify Heroku plugin for v2#1642
hermanschaaf wants to merge 55 commits intocloudquery:feat/cloudquery-cli-v2from
hermanschaaf:feat/cloudquery-cli-v2-heroku

Conversation

@hermanschaaf
Copy link
Copy Markdown
Contributor

This serves as an example of the changes necessary to migrate a plugin to v2

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:

cloudquery#1104
cloudquery#1055
cloudquery#983
cloudquery#859
cloudquery#858
cloudquery#888
cloudquery#857
cloudquery#854
cloudquery#904
cloudquery#751
cloudquery#539
cloudquery#392
cloudquery#292
cloudquery#159
cloudquery#908
cloudquery#840
@hermanschaaf hermanschaaf changed the title WIP: Modify Heroku plugin for v2 feat: Modify Heroku plugin for v2 Sep 6, 2022
@hermanschaaf hermanschaaf marked this pull request as ready for review September 6, 2022 07:33
@hermanschaaf hermanschaaf requested review from a team and disq and removed request for a team September 6, 2022 07:33
* update to latest sdk

* feat: Move to new SDK

* more fixes to autogeneration

* more fixes to gcp

* feat: Add sentry

* remove old faker
@hermanschaaf hermanschaaf requested review from yevgenypats and removed request for disq September 6, 2022 07:36
@yevgenypats yevgenypats force-pushed the feat/cloudquery-cli-v2 branch from 56f2203 to 7f3548d Compare September 6, 2022 09:33
@yevgenypats yevgenypats deleted the branch cloudquery:feat/cloudquery-cli-v2 September 6, 2022 09:44
@yevgenypats yevgenypats closed this Sep 6, 2022
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