Skip to content

Consolidate API key and base URL property resolution to avoid code duplication#5376

Open
nicolaskrier wants to merge 1 commit intospring-projects:mainfrom
nicolaskrier:mistral-ai-default-properties
Open

Consolidate API key and base URL property resolution to avoid code duplication#5376
nicolaskrier wants to merge 1 commit intospring-projects:mainfrom
nicolaskrier:mistral-ai-default-properties

Conversation

@nicolaskrier
Copy link
Copy Markdown
Contributor

@nicolaskrier nicolaskrier commented Jan 30, 2026

  • Introduce getApiKeyOrDefaultFrom and getBaseUrlOrDefaultFrom in MistralAiParentProperties,
  • Add comprehensive unit tests for these methods,
  • Replace duplicated logic in Mistral AI auto-configurations by using these methods.

@nicolaskrier nicolaskrier force-pushed the mistral-ai-default-properties branch 2 times, most recently from 99635d4 to ca351a5 Compare February 1, 2026 21:33
@nicolaskrier nicolaskrier marked this pull request as ready for review February 1, 2026 21:35
@nicolaskrier nicolaskrier force-pushed the mistral-ai-default-properties branch from ca351a5 to 38f719c Compare February 22, 2026 13:30
@nicolaskrier nicolaskrier force-pushed the mistral-ai-default-properties branch from 38f719c to 87c853a Compare March 7, 2026 13:46
@nicolaskrier nicolaskrier force-pushed the mistral-ai-default-properties branch 2 times, most recently from e4b8396 to 9ede39c Compare March 21, 2026 15:20
- Introduce getApiKeyOrDefaultFrom and getBaseUrlOrDefaultFrom in MistralAiParentProperties
- Add comprehensive unit tests for these methods
- Replace duplicated logic in Mistral AI auto-configurations by using these methods

Signed-off-by: Nicolas Krier <7557886+nicolaskrier@users.noreply.github.com>
@nicolaskrier nicolaskrier force-pushed the mistral-ai-default-properties branch from 9ede39c to b312d2b Compare April 18, 2026 11:09
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.

1 participant