ioBroker.js-controller icon indicating copy to clipboard operation
ioBroker.js-controller copied to clipboard

Out of callback hell for alias methods

Open foxriver76 opened this issue 1 year ago • 6 comments

  • refactored to remove lots of lines by porting checkStates to async
  • side effect internal _setState and _getForeignState now can fully return the result as promise via maybeCb

foxriver76 avatar Sep 23 '22 07:09 foxriver76

This pull request introduces 2 alerts when merging 48082e69c6a4fb7ffd68efab82adffe6e78cdcb3 into f93f2710227180bf1f5bd79f6bce3fd358eebb09 - view on LGTM.com

new alerts:

  • 2 for Useless conditional

lgtm-com[bot] avatar Sep 23 '22 07:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 6aa3a63530293302f3084abd6e11c21668dbed79 into f93f2710227180bf1f5bd79f6bce3fd358eebb09 - view on LGTM.com

new alerts:

  • 2 for Useless conditional

lgtm-com[bot] avatar Sep 23 '22 08:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 76e9ba28eedd3e057d5f5ab73608ebcc79ba3915 into f93f2710227180bf1f5bd79f6bce3fd358eebb09 - view on LGTM.com

new alerts:

  • 2 for Useless conditional

lgtm-com[bot] avatar Sep 23 '22 08:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 93ae543de315dac3e566bde91c989f0f1c81ea4b into f93f2710227180bf1f5bd79f6bce3fd358eebb09 - view on LGTM.com

new alerts:

  • 2 for Useless conditional

lgtm-com[bot] avatar Sep 23 '22 09:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 2e52823d30152c1ce40c7a6df44ef1be75d0c735 into f93f2710227180bf1f5bd79f6bce3fd358eebb09 - view on LGTM.com

new alerts:

  • 2 for Useless conditional

lgtm-com[bot] avatar Sep 23 '22 10:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging f717e5c8531869c4a74e16c0251b47afffd2374a into f93f2710227180bf1f5bd79f6bce3fd358eebb09 - view on LGTM.com

new alerts:

  • 2 for Useless conditional

lgtm-com[bot] avatar Sep 23 '22 10:09 lgtm-com[bot]