Fix instance capabilities in report#6080
Conversation
We create a `Set` when loading the config (to avoid costly looking), but this must not modify the capabilities which are returned by reportHandler. Issue: CLDSRV-850
Hello francoisferrand,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Issue: CLDSRV-850
Codecov Report❌ Patch coverage is
Additional details and impacted files
... and 2 files with indirect coverage changes @@ Coverage Diff @@
## development/9.1 #6080 +/- ##
===================================================
+ Coverage 83.77% 83.79% +0.02%
===================================================
Files 191 191
Lines 12350 12350
===================================================
+ Hits 10346 10349 +3
+ Misses 2004 2001 -3
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Request integration branchesWaiting for integration branch creation to be requested by the user. To request integration branches, please comment on this pull request with the following command: Alternatively, the |
|
/approve |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
The following options are set: approve |
|
/bypass_peer_approval |
|
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
Please check the status of the associated issue CLDSRV-850. Goodbye francoisferrand. The following options are set: bypass_peer_approval, approve |
We create a
Setwhen loading the config (to avoid costly looking),but this must not modify the capabilities which are returned by
reportHandler.
Issue: CLDSRV-850