Skip to content

Comments

refactor: rename output fields to better match their purpose, add pending linux reboots#267

Merged
rwaffen merged 5 commits intovoxpupuli:mainfrom
rwaffen:update_outputs
Feb 19, 2026
Merged

refactor: rename output fields to better match their purpose, add pending linux reboots#267
rwaffen merged 5 commits intovoxpupuli:mainfrom
rwaffen:update_outputs

Conversation

@rwaffen
Copy link
Member

@rwaffen rwaffen commented Jan 22, 2026

No description provided.

@rwaffen rwaffen changed the title refactor: rename skipped to health_check_failed, add pendeing linux reboots refactor: rename output fields to better match their purpose, add pending linux reboots Jan 22, 2026
@rwaffen
Copy link
Member Author

rwaffen commented Jan 22, 2026

dang it, need to clean up the code a bit to make rubocop happy

@rwaffen rwaffen added the backwards-incompatible This change will lead to a major version bump for the next release label Jan 23, 2026
}.map |$n| { $n.target }

$with_patches = $patching_result.ok_set.filter |$item| {
! $item.value['packages_updated'].empty
Copy link
Member

Choose a reason for hiding this comment

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

ah neat. Didn't know you can use ! as an expression here

Copy link
Member Author

Choose a reason for hiding this comment

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

oh yes, in a filter or any block you can do this, it just reverses the boolean returned from the .empty method

@rwaffen rwaffen merged commit 1fa56b8 into voxpupuli:main Feb 19, 2026
14 checks passed
@rwaffen rwaffen deleted the update_outputs branch February 19, 2026 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backwards-incompatible This change will lead to a major version bump for the next release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants