Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: cloudquery/cloudquery
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: cli-v1.5.9
Choose a base ref
...
head repository: cloudquery/cloudquery
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: cli-v2.0.0
Choose a head ref
  • 3 commits
  • 8 files changed
  • 2 contributors

Commits on Nov 8, 2022

  1. chore: Update CLI version to v1.5.9 (#3871) 

    Updates the CLI latest version to v1.5.9
    @cq-bot
    cq-bot authored Nov 8, 2022
    Configuration menu
    Copy the full SHA
    c3a5ae9 View commit details
    Browse the repository at this point in the history

  2. fix!: Make path a required config parameter (#3872) 

    This SDK upgrade to the CLI adds the code in cloudquery/plugin-sdk#371, and as such will make `path` a required config parameter. We consider this a breaking change, so the CLI version will be bumped to 2.0.0. 

As context, we are making `path` required (and not inferred from `name`) because there has been some confusion around these settings. Previously `path` could have been inferred from `name` in some circumstances, but when using multiple configs, this was not always possible, causing confusion.

From now on, `path` is a required parameter. It will usually take the form of `cloudquery/aws`, where `aws` is the source plugin name in this example. For the postgresql destination, it will be `cloudquery/postgresql`.

- `name` should be thought of as the unique identifier of the particular source/destination config. It needs to be unique, as it will be used for overwriting stale entries in `overwrite-delete-stale` mode, inserted as a column alongside all entries, and ensure there are no race conditions when running multiple configs in parallel.
- `path` is normally the unique path to download the plugin from. We have some special conventions around this, documented [here](https://www.cloudquery.io/docs/developers/creating-new-plugin#naming-conventions). It can also be used during development to point to a locally-running binary or GRPC server.

Related:
 - cloudquery/plugin-sdk#317
 - cloudquery/plugin-sdk#256
    @hermanschaaf
    hermanschaaf authored Nov 8, 2022
    Configuration menu
    Copy the full SHA
    b0b4560 View commit details
    Browse the repository at this point in the history

  3. chore(main): Release cli v2.0.0 (#3873) 

    🤖 I have created a release *beep* *boop*
---


## [2.0.0](cli-v1.5.9...cli-v2.0.0) (2022-11-08)


### ⚠ BREAKING CHANGES

* Make path a required config parameter (#3872)

### Fixes

* Make path a required config parameter ([#3872](#3872)) ([b0b4560](b0b4560))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
    @cq-bot
    cq-bot authored Nov 8, 2022
    Configuration menu
    Copy the full SHA
    71d53fc View commit details
    Browse the repository at this point in the history
Loading