Skip to content
This repository has been archived by the owner on May 3, 2022. It is now read-only.

[WIP] Well known custom actions #703

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

[WIP] Well known custom actions #703

wants to merge 2 commits into from

Conversation

silvin-lubecki
Copy link
Contributor

@silvin-lubecki silvin-lubecki commented Apr 8, 2019 •
edited
Loading

  • Status is no more a mandatory action, but it is now a well known custom action
  • Downgrade has been totally removed has it wasn't really implemented in duffle

⚠️ Do not merge, waiting for spf13/viper#673 to be merged, fixing an issue where viper can't deserialize a key which has dots in it (like io.cnab.status).

@silvin-lubecki
Remove out-of-date downgrade action references in the code comments, …
d98ca65 
…documentation and examples.

Signed-off-by: Silvin Lubecki <[email protected]>
@silvin-lubecki
According to the cnab specs, Status is no more a mandatory action, bu…
712c008 
…t a "well known custom action".

Code, tests, documentation and comments were updated to reflect this change.

Signed-off-by: Silvin Lubecki <[email protected]>
@silvin-lubecki silvin-lubecki changed the title Well known custom actions [WIP] Well known custom actions Apr 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@silvin-lubecki