Closed chrisjsewell closed 2 years ago
Base: 90.46% // Head: 90.46% // Increases project coverage by +0.01%
:tada:
Coverage data is based on head (
e4efc9e
) compared to base (8c7640b
). Patch coverage: 100.00% of modified lines in pull request are covered.
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
and obviously I tested this fix works, in the context of VS Code:
(this does not work currently)
and obviously I tested this fix works, in the context of VS Code:
How does this show that it is working? The to_context
method is not decorated with auto_persist
is it?
The to_context method is not decorated with auto_persist is it?
No, the whole WorkChain
class is decorated with auto_persist
How does this show that it is working?
This is it without the change, all it knows is that wc
is a Savable
This is used in aiida-core, to decorate the
WorkChain
class. The problem currently is that, because of the decorator typing, no methods defined on subclasses ofSavable
are available for static analysers (such as IDE auto-completions). This change ensures the output of the decorator is the same as the input.