Skip to content

Remove unused acceptance MergeBundleConfig helper - #6644

Open
denik wants to merge 1 commit into
mainfrom
denik/rm-mergebundleconfig
Open

Remove unused acceptance MergeBundleConfig helper#6644
denik wants to merge 1 commit into
mainfrom
denik/rm-mergebundleconfig

Conversation

@denik

@denik denik commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

MergeBundleConfig was added in #2809 (May 2025) to back an acceptance BundleConfig setting, but it's never invoked — nothing outside its own test calls it. Only bundle_config_test.go references it, so deadcode -test ./... doesn't flag it.

This pull request and its description were written by Isaac.

Added in #2809 (May 2025) to back an acceptance BundleConfig setting, but it
is never invoked: nothing outside its own test calls MergeBundleConfig. Only
bundle_config_test.go references it, so deadcode -test never flagged it.

Co-authored-by: Isaac <no-reply@databricks.com>
@eng-dev-ecosystem-bot

Copy link
Copy Markdown
Collaborator

Integration test report

Commit: ab16b04

Run: 34609668656

Env ✅​pass 🙈​skip Time
✅​ aws linux 276 15 8:48
✅​ aws windows 278 13 5:33
✅​ azure linux 275 15 9:20
✅​ azure windows 277 13 5:08
✅​ gcp linux 276 15 8:29
✅​ gcp windows 278 13 4:51
Top 3 slowest tests (at least 2 minutes):
duration env testname
5:13 aws windows TestAccept
5:07 azure windows TestAccept
3:33 gcp windows TestAccept

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants