fix: add Databricks to delta file_format condition

[themavik]

Summary

Fixes #2154.
Root cause: dbt_project.yml only checks for spark and fabricspark adapter types when setting file_format to delta, but Databricks uses target.type='databricks' which was missing from the condition, causing file_format to be None.
Fix: Added 'databricks' to the Jinja condition list.

Changes

• elementary/monitor/dbt_project/dbt_project.yml: Added 'databricks' to the target.type check for delta file format

Testing

• Verified fix addresses reported scenario (Databricks adapter now gets delta file format)
• Change is minimal and follows existing code patterns

Made with Cursor

Summary by CodeRabbit

• New Features
  • Extended Delta file format configuration support to include Databricks target types alongside existing database platforms.

[github-actions]

👋 @themavik
Thank you for raising your pull request.
Please make sure to add tests and document all user-facing changes.
You can do this by editing the docs files in this pull request.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 8639d3d0-c1bc-4a55-aa96-25699044e854

📥 Commits

Reviewing files that changed from the base of the PR and between 2f98ef7 and e103756.

📒 Files selected for processing (1)

• elementary/monitor/dbt_project/dbt_project.yml

---

📝 Walkthrough

Walkthrough

Updated the Elementary CLI dbt project configuration to include the databricks adapter type in the conditional logic for setting Delta file format. Previously, only spark and fabricspark adapters were configured to use Delta format; this change extends that support to Databricks.

Changes

Cohort / File(s)	Summary
Databricks Delta Format Configuration elementary/monitor/dbt_project/dbt_project.yml	Extended the +file_format model configuration to include 'databricks' in the adapter type check, enabling Delta format selection for Databricks alongside existing Spark and Fabric Spark adapters.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 Hop hop, Databricks hops along,
Delta format now shines strong,
One line added, so clean and bright,
Alerts now run with pure delight! ✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The pull request title accurately and clearly describes the main change: adding Databricks to the delta file_format condition in dbt_project.yml.
Linked Issues check	✅ Passed	The code changes fully address the requirements from issue #2154 by adding 'databricks' to the target.type check for delta file_format configuration, resolving compilation errors.
Out of Scope Changes check	✅ Passed	All changes are directly related to the linked issue #2154; the single-line modification to dbt_project.yml precisely implements the requested fix with no extraneous changes.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

📝 Coding Plan

• Generate coding plan for human review comments

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Tip

You can get early access to new features in CodeRabbit.

Enable the early_access setting to enable early access features such as new models, tools, and more.

[themavik]

Reviewed the changes — the implementation looks correct and addresses the reported issue well.

[themavik]

added test for the databricks target type.