search

eclipse / microprofile-health

microprofile-health
Apache License 2.0
107 stars 61 forks source link

Refactoring HealthCheckResponse to remove Optional from the data field #332

Closed fabiobrz closed 2 months ago

fabiobrz commented 2 months ago

Quick attempt at refactoring HealthCheckResponse to use just Map<> as the data type for the data field. This is a breaking change, since the ctor is affected, hence the microprofile-health-api version is bumped as well.

~Draft, since the changes require #331 (rebased on it temporarily)~

Fixes #323

FYI @xstefank

fabiobrz commented 2 months ago

Hi @pgunapal - @xstefank approved yesterday, could you have a look to the changes as well?

fabiobrz commented 2 months ago

Looks good, please update the copyright years for the files.

Thanks @pgunapal, fixed.