Skip to content

[test-only] Fork PR #5904#5930

Closed
janniklasrose wants to merge 3 commits into
mainfrom
fork-pr/5904
Closed

[test-only] Fork PR #5904#5930
janniklasrose wants to merge 3 commits into
mainfrom
fork-pr/5904

Conversation

@janniklasrose

Copy link
Copy Markdown
Contributor

Run tests from fork contribution (#5904)

mvanhorn and others added 2 commits July 13, 2026 02:36
…source

Inside a bundle root the resolved config labels every attribute with the
"bundle" source, so a profile selected via DATABRICKS_CONFIG_PROFILE was
rendered as "(from bundle)" instead of "(from DATABRICKS_CONFIG_PROFILE
environment variable)". Extend getAuthDetails to detect the env var, mirroring
the flag > env precedence in cmd/root/bundle.go:getProfile. The value guard
keeps genuine bundle-file profiles reading "(from bundle)".

Closes #2303

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01M1ZTEJFKv3BAW36VmBwe2v
@github-actions

Copy link
Copy Markdown
Contributor

Approval status: pending

/cmd/auth/ - needs approval

Files: cmd/auth/describe.go
Suggested: @simonfaltum
Also eligible: @renaudhartert-db, @hectorcast-db, @parthban-db, @tanmay-db, @Divyansh-db, @tejaskochar-db, @mihaimitrea-db, @chrisst, @rauchy

General files (require maintainer)

6 files changed
Based on git history:

  • @simonfaltum -- recent work in cmd/auth/

Any maintainer (@andrewnester, @anton-107, @denik, @pietern, @shreyas-goenka, @simonfaltum, @renaudhartert-db, @lennartkats-db) can approve all areas.
See OWNERS for ownership rules.

@janniklasrose

Copy link
Copy Markdown
Contributor Author

for test purposes only, #5904 merged

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants