Skip to content

fix: use git tag --no-column to prevent dbt deps breakage with columnar git output#12728

Open
claygeo wants to merge 2 commits into
dbt-labs:mainfrom
claygeo:fix/git-tag-no-column
Open

fix: use git tag --no-column to prevent dbt deps breakage with columnar git output#12728
claygeo wants to merge 2 commits into
dbt-labs:mainfrom
claygeo:fix/git-tag-no-column

Conversation

@claygeo
Copy link
Copy Markdown
Contributor

@claygeo claygeo commented Mar 29, 2026

Problem

Fixes #10381

When a user configures git with column.ui=always (or any columnar output setting), git tag --list outputs tags in multi-column format. dbt's list_tags() splits by newline and gets garbled tag names, breaking dbt deps.

Fix

Replace git tag --list with git tag --no-column in core/dbt/clients/git.py. The --no-column flag explicitly disables columnar output regardless of user git configuration. Available since git 1.7.11 (2012).

As suggested in the issue by @dbeatty10 (dbt maintainer).

1 line changed.

@claygeo claygeo requested a review from a team as a code owner March 29, 2026 02:52
@cla-bot cla-bot Bot added the cla:yes label Mar 29, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide.

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide.

@github-actions github-actions Bot added the community This PR is from a community member label Mar 29, 2026
claygeo added a commit to claygeo/dbt-core that referenced this pull request Mar 30, 2026
@claygeo claygeo force-pushed the fix/git-tag-no-column branch from 6db3ff8 to 563934c Compare March 30, 2026 15:27
claygeo added a commit to claygeo/dbt-core that referenced this pull request Mar 30, 2026
@claygeo claygeo force-pushed the fix/git-tag-no-column branch from 563934c to f4c2219 Compare March 30, 2026 17:28
claygeo added a commit to claygeo/dbt-core that referenced this pull request Mar 30, 2026
@claygeo claygeo force-pushed the fix/git-tag-no-column branch from f4c2219 to 1768960 Compare March 30, 2026 18:29
claygeo added a commit to claygeo/dbt-core that referenced this pull request Mar 30, 2026
@claygeo claygeo force-pushed the fix/git-tag-no-column branch from 1768960 to d495263 Compare March 30, 2026 18:38
claygeo added 2 commits April 1, 2026 11:21
…ar git output

When a user configures git with column.ui=always, git tag --list
outputs tags in columnar format which dbt cannot parse, breaking
dbt deps. Use --no-column to explicitly disable columnar output
regardless of user git configuration.

The --no-column flag has been available since git 1.7.11 (2012).

Fixes dbt-labs#10381
@claygeo claygeo force-pushed the fix/git-tag-no-column branch from d495263 to de2221e Compare April 1, 2026 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla:yes community This PR is from a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use git tag --no-column for listing tags for dbt deps

1 participant