Skip to content

Track action in snapshots#1399

Merged
matheuscscp merged 1 commit intomainfrom
snapshot-action
Feb 2, 2026
Merged

Track action in snapshots#1399
matheuscscp merged 1 commit intomainfrom
snapshot-action

Conversation

@matheuscscp
Copy link
Member

Introduce .action field for entries in .status.history.

Signed-off-by: Matheus Pimenta <matheuscscp@gmail.com>
Comment on lines +70 to +73
// Action is the action that resulted in this observation.
// It's not serialized to JSON because it's not needed for
// the digest. The action is only tracked for visibility.
Action v2.ReleaseAction `json:"-"`
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this field is purely a visibility feature I'm avoiding adding it to the digest calculation for now, a ton of tests failed adding it to the digest.

Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Great UX improvment 💯

@matheuscscp matheuscscp merged commit 94a3c03 into main Feb 2, 2026
5 checks passed
@matheuscscp matheuscscp deleted the snapshot-action branch February 2, 2026 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/api API related issues and pull requests enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants