Skip to content

Correctly resolve relative includes in databricks.yml from parent folders#1870

Open
misha-db wants to merge 5 commits into
mainfrom
databricks-yml-include-section
Open

Correctly resolve relative includes in databricks.yml from parent folders#1870
misha-db wants to merge 5 commits into
mainfrom
databricks-yml-include-section

Conversation

@misha-db
Copy link
Copy Markdown
Contributor

Changes

Correctly resolve relative includes in databricks.yml from parent folders

Closes: #1818
Closes: #1731

Tests

Unit tests in BundleFileSet.test.ts

@misha-db misha-db requested a review from ilia-db March 30, 2026 08:41
@misha-db misha-db marked this pull request as ready for review March 30, 2026 08:41
return bundle.include;
}

async getIncludedFilesGlob() {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it used anywhere now?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not used, removed

@github-actions
Copy link
Copy Markdown
Contributor

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/vscode

Inputs:

  • PR number: 1870
  • Commit SHA: a9951dcf1140f26446af2e6f19c3194e6057f13e

Checks will be approved automatically on success.

@misha-db misha-db requested a review from ilia-db April 17, 2026 13:05
@ranasalalali
Copy link
Copy Markdown

ranasalalali commented May 17, 2026

Friendly nudge on this one — this fix for relative includes in databricks.yml from parent folders would be really useful. It looks like Linux is passing, Windows is failing, and integration tests may need the manual trigger. If someone from the team can take a look or help get the remaining checks unstuck, that’d be appreciated.

@mjestertea
Copy link
Copy Markdown

Hello

This addition would be a great to fix an issue with the functionality for many of the bundles

Thank you!

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

Labels

None yet

Projects

None yet

4 participants