Skip to content

Conversation

@OlgaMaciaszek
Copy link
Collaborator

See: #813.

@OlgaMaciaszek OlgaMaciaszek added the enhancement New feature or request label Feb 8, 2023
@OlgaMaciaszek OlgaMaciaszek added this to the 4.0.2 milestone Feb 8, 2023
@OlgaMaciaszek OlgaMaciaszek self-assigned this Feb 8, 2023
@OlgaMaciaszek OlgaMaciaszek marked this pull request as draft February 8, 2023 14:34
@OlgaMaciaszek OlgaMaciaszek changed the title DRAFT/ DO NOT MERGE YET: Property based url configurable at runtime in AOT/ native Property based url configurable at runtime in AOT/ native Feb 13, 2023
@OlgaMaciaszek OlgaMaciaszek marked this pull request as ready for review February 13, 2023 15:12
@codecov
Copy link

codecov bot commented Feb 13, 2023

Codecov Report

Merging #827 (ad566f1) into main (da15912) will increase coverage by 0.02%.
The diff coverage is 85.71%.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #827      +/-   ##
============================================
+ Coverage     74.94%   74.96%   +0.02%     
- Complexity      575      577       +2     
============================================
  Files            67       68       +1     
  Lines          2239     2245       +6     
  Branches        302      302              
============================================
+ Hits           1678     1683       +5     
  Misses          394      394              
- Partials        167      168       +1     
Impacted Files Coverage Δ
...framework/cloud/openfeign/PropertyBasedTarget.java 83.33% <83.33%> (ø)
...mework/cloud/openfeign/FeignClientFactoryBean.java 74.57% <100.00%> (ø)

@OlgaMaciaszek OlgaMaciaszek merged commit 54a6a56 into main Feb 14, 2023
@OlgaMaciaszek OlgaMaciaszek removed this from the 4.0.2 milestone Feb 20, 2023
@OlgaMaciaszek OlgaMaciaszek linked an issue Feb 20, 2023 that may be closed by this pull request
@oleh-hudyma
Copy link

@OlgaMaciaszek when can we expect for this to be released?

@OlgaMaciaszek
Copy link
Collaborator Author

@OlegHudyma a release is scheduled for later this week.

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support Dynamic URL In Native Image

3 participants