Skip to content

Improve duplicate macro error message - #1647

Merged
dbeatty10 merged 3 commits into
mainfrom
dbeatty/issue-12484
Feb 14, 2026
Merged

Improve duplicate macro error message#1647
dbeatty10 merged 3 commits into
mainfrom
dbeatty/issue-12484

Conversation

@dbeatty10

@dbeatty10 dbeatty10 commented Feb 13, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

resolves #1648

Problem

When there are duplicated macros, the error message assumes there is only a single duplicate (rather than possibly multiple).

Solution

Improve the error message.

Checklist

  • I have read the contributing guide and understand what's expected of me
  • I have run this code in development and it appears to resolve the stated issue
  • This PR includes tests, or tests are not required/relevant for this PR
  • This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

@cla-bot cla-bot Bot added the cla:yes The PR author has signed the CLA label Feb 13, 2026
@cla-bot
cla-bot Bot temporarily deployed to dbt-snowflake February 13, 2026 17:32 Inactive
@dbeatty10
dbeatty10 marked this pull request as ready for review February 13, 2026 18:26
@dbeatty10
dbeatty10 requested a review from a team as a code owner February 13, 2026 18:26
@dbeatty10
dbeatty10 enabled auto-merge (squash) February 13, 2026 23:56
@dbeatty10
dbeatty10 merged commit 67852cd into main Feb 14, 2026
74 of 76 checks passed
@dbeatty10
dbeatty10 deleted the dbeatty/issue-12484 branch February 14, 2026 01:06
@colin-k-rogers colin-k-rogers mentioned this pull request Feb 25, 2026
Closed
3 tasks
@github-actions

github-actions Bot commented Mar 2, 2026

Copy link
Copy Markdown
Contributor

❌ Promotion to stable branch failed. Please check the promotion job for details.

You will need to manually promote the changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@QMalcolm QMalcolm QMalcolm approved these changes

Assignees

No one assigned

Labels

cla:yes The PR author has signed the CLA promote stable

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Feature] Clarify error when file(s) contain duplicate macros

4 participants

@dbeatty10 @QMalcolm @colin-k-rogers @Vigneshjayabalan