Skip to content

Call TypeAwareExpressionEvaluator.evaluate(String,Class) if available - #161

Merged
kwin merged 1 commit into
mainfrom
feature/support-TypeAwareExpressionEvaluator-in-configuratonconverter
Sep 30, 2024
Merged

kwin merged 1 commit into
mainfrom
feature/support-TypeAwareExpressionEvaluator-in-configuratonconverter

Conversation

@kwin

@kwin kwin commented Aug 29, 2024

Copy link
Copy Markdown
Contributor

Only enforce types in some places (where conversion afterwards never happens)

This closes #160

@kwin
kwin requested review from cstamas and mcculls August 29, 2024 16:02
@kwin
kwin force-pushed the feature/support-TypeAwareExpressionEvaluator-in-configuratonconverter branch 2 times, most recently from 98f5162 to 92a3066 Compare August 29, 2024 16:09
Only enforce types in some places (where conversion afterwards never
happens)

This closes #160
@kwin
kwin force-pushed the feature/support-TypeAwareExpressionEvaluator-in-configuratonconverter branch from 92a3066 to 77e0db5 Compare August 29, 2024 16:12
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
62.3% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

@kwin

kwin commented Sep 21, 2024

Copy link
Copy Markdown
Contributor Author

I am gonna merge beginning of next week if there are no concerns with this.

@kwin
kwin merged commit 8bce6fe into main Sep 30, 2024
@kwin
kwin deleted the feature/support-TypeAwareExpressionEvaluator-in-configuratonconverter branch February 11, 2026 10:54
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.

Support TypeAwareExpressionEvaluator in AbstractConfigurationConverter

1 participant