Skip to content

Conversation

@MichelleArk
Copy link
Contributor

In tandem with dbt-labs/dbt-common#323, support DBT_ENGINE prefix for recorder-related env vars

@MichelleArk MichelleArk requested a review from a team as a code owner November 5, 2025 21:49
@cla-bot cla-bot bot added the cla:yes label Nov 5, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 5, 2025

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.

@codecov
Copy link

codecov bot commented Nov 5, 2025

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 91.93%. Comparing base (1bd81f5) to head (452aad4).
⚠️ Report is 4 commits behind head on main.

❌ Your patch status has failed because the patch coverage (0.00%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #12149      +/-   ##
==========================================
+ Coverage   91.92%   91.93%   +0.01%     
==========================================
  Files         203      203              
  Lines       24846    24846              
==========================================
+ Hits        22839    22843       +4     
+ Misses       2007     2003       -4     
Flag Coverage Δ
integration 88.87% <0.00%> (+0.08%) ⬆️
unit 65.16% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
Unit Tests 65.16% <0.00%> (ø)
Integration Tests 88.87% <0.00%> (+0.08%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@QMalcolm QMalcolm merged commit 001e729 into main Nov 6, 2025
56 of 58 checks passed
@QMalcolm QMalcolm deleted the record-engine-env-vars branch November 6, 2025 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants