Our changelog entries are great for end-user facing changelogs, but for programmatic consumers of dbt-adapters (e.g. dbt-core) -- they do not always accurately signal what kind of interface changes are occurring and possibly breaking. Ideally, this is communicated via the versioning of dbt-common, but the person releasing dbt-common needs to know what version to release!
To that end, we'd like to introduce a lightweight CI check that would be required on any PR to dbt-adapters. The CI check would require one of release_type: major, release_type: minor, release_type: patch labels to be applied to the PR prior to merging. That way, the person releasing dbt-common could pick the highest release type among a set of changes and infer the version to release of dbt-common.
Context
No response
Acceptance criteria
Introduce a required CI check on PRs that passes if one of release_type: major, release_type: minor, release_type: patch labels exists, and fails otherwise
Housekeeping
dbt-adapters
Short description
Our changelog entries are great for end-user facing changelogs, but for programmatic consumers of dbt-adapters (e.g. dbt-core) -- they do not always accurately signal what kind of interface changes are occurring and possibly breaking. Ideally, this is communicated via the versioning of dbt-common, but the person releasing dbt-common needs to know what version to release!
To that end, we'd like to introduce a lightweight CI check that would be required on any PR to dbt-adapters. The CI check would require one of release_type: major, release_type: minor, release_type: patch labels to be applied to the PR prior to merging. That way, the person releasing dbt-common could pick the highest release type among a set of changes and infer the version to release of dbt-common.
Context
No response
Acceptance criteria
Introduce a required CI check on PRs that passes if one of release_type: major, release_type: minor, release_type: patch labels exists, and fails otherwise
Testing
Manual testing for CI check
Security
N/A
Docs
N/A
Consequences
N/A