Code review comment for ~cjwatson/launchpad:charm-scripts-actions

Revision history for this message
Ines Almeida (ines-almeida) wrote :

Makes sense to me

Added a small question in the comments.
Also what's the purpose of `hookenv.action_set()`? In the docs it seems to be the value that the action returns - but we don't seem to use any return value from the actions (?). Is it mostly because as action needs to return something?

review: Approve

« Back to merge proposal