chore: add Release Please workflow and ADR issue template#55
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
Two things that compose with the existing release pipeline and issue templates without changing user-facing behaviour:
.github/ISSUE_TEMPLATE/adr.yml) — captures the shape used by the W025 and dbt-aware rule pack ADRs (What/Why/Scope/Compatibility/Plan/Open questions/Rejected). Stops the next architecture proposal from being typed by hand into a blank textarea.release-please-config.json+.release-please-manifest.json(pinned atv0.7.0) +.github/workflows/release-please.yml. Watchesmainfor Conventional Commits, opens a release PR with version bump + CHANGELOG promotion, and on merge cuts av*.*.*tag. The existingrelease.ymlalready triggers on tags and publishes to PyPI via Trusted Publishing — no change there.Related issue
None. Triggered by external review feedback on contributor-friction and release-chore overhead.
If this adds or changes a rule
N/A — no rule changes.
Tests
pytest -qpasses (254 passed / 1 skipped, unchanged from main)ruff check .passesThis PR only touches
.github/and root-level config — no test changes expected.Checklist
chore,ci)CHANGELOG.mdentry — infra/governance only, not user-facing behaviourAnything else reviewers should know
What the first Release Please PR will look like. After this merges, RP will sweep up every Conventional Commit since
v0.7.0— currently that's W024, the S001 fix, E009, T006, plus anything else that has landed. Two decisions on that release PR:[Unreleased]section has hand-written prose with more context than commit subjects carry. Keep the prose (delete RP's terse entries) or accept RP's entries (delete the prose). I'd keep the prose.feature/W025-assertion-malformedandfeature/dbt-pack-scaffoldland before this, they go into the first auto-cut release. If this lands first, the auto-cut release is chore-only (v0.7.1). Recommended order: W025 → DBT001 → this branch last.Permissions. The workflow declares
contents: writeandpull-requests: writeforrelease-please-action. No new secrets; usesGITHUB_TOKEN.Reversibility. If RP misbehaves, deleting
.github/workflows/release-please.ymlcleanly disables it. The manifest + config can stay.