Skip to content

OAK-12298: SystemPropertySupplier: allow to signal that property is not present (implies default value)#3044

Closed
reschke wants to merge 3 commits into
trunkfrom
OAK-12298b
Closed

OAK-12298: SystemPropertySupplier: allow to signal that property is not present (implies default value)#3044
reschke wants to merge 3 commits into
trunkfrom
OAK-12298b

Conversation

@reschke

@reschke reschke commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

I'm starting to think that the API change is too large, will try to figure out something simpler.

@reschke reschke self-assigned this Jul 20, 2026
@github-actions

github-actions Bot commented Jul 20, 2026

Copy link
Copy Markdown

Commit-Check ❌

Commit rejected by Commit-Check.                                  
                                                                  
  (c).-.(c)    (c).-.(c)    (c).-.(c)    (c).-.(c)    (c).-.(c)  
   / ._. \      / ._. \      / ._. \      / ._. \      / ._. \   
 __\( C )/__  __\( H )/__  __\( E )/__  __\( C )/__  __\( K )/__ 
(_.-/'-'\-._)(_.-/'-'\-._)(_.-/'-'\-._)(_.-/'-'\-._)(_.-/'-'\-._)
   || E ||      || R ||      || R ||      || O ||      || R ||   
 _.' '-' '._  _.' '-' '._  _.' '-' '._  _.' '-' '._  _.' '-' '._ 
(.-./`-´\.-.)(.-./`-´\.-.)(.-./`-´\.-.)(.-./`-´\.-.)(.-./`-´\.-.)
 `-´     `-´  `-´     `-´  `-´     `-´  `-´     `-´  `-´     `-´ 
                                                                  
Commit rejected.                                                  
                                                                  
Type subject_max_length check failed ==> OAK-12298: SystemPropertySupplier: allow to signal that property is not present (implies default value) - fix class assignable checks 
Subject must be at most 120 characters
Suggest: Keep the subject concise (<= configured max)

…ot present (implies default value) - fix class assignable checks
@reschke reschke closed this Jul 20, 2026
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

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

See analysis details on SonarQube Cloud

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.

2 participants