Benoit Perigaud

Approach to handle setting up license mapping in different resources

4

The current [`dbtcloud_license_map` resource](https://registry.terraform.io/providers/dbt-labs/dbtcloud/latest/docs/resources/license_map) can only be set once for an entire dbt Cloud account This is because the identifier of a resource map is unique for an account rather...

Rename `dbtcloud_project_artefacts` to `dbtcloud_project_artifacts`

3

We should stick to the American spelling

Issue when linking GitLab using Service Tokens

7

When using service tokens to call the API to link a GitLab repo natively with dbt Cloud the connection is not set up correctly and require manually reconnecting the repo....

Some actions can take longer than other, e.g. when we have to do one API call per job to get the env var overrides. This change is a POC of...

Is there an opportunity to create a fixture that returns config dicts? Might be way to reduce duplicate (or nearly duplicate) code. _Originally posted by @nicholasyager in https://github.com/dbt-labs/dbt-jobs-as-code/pull/37#discussion_r1054562848_

Improve overall performance of the tool

1

In some scenarios, we perform multiple HTTP requests to dbt Cloud. `import-jobs` requires one call per job for example in order to get the env-var overwrites at the job level....

When the config is changed provide more information about the value before/after

1

There's room in the future for this to be a little more detailed (e.g. what was the value before, and what is it now?). This is a great start 👍🏻...

Followup of #23 for checking that env vars listed already exist in dbt Cloud

More of a long-term architectural comment: This logic is very sound, and could likely be abstracted into validation functions that can be composed easily. Something to consider for a MUCH...