Skip to content

Top down evaluation custom promise type test#3029

Merged
larsewi merged 1 commit into
cfengine:masterfrom
victormlg:top-down-evaluation
Aug 21, 2025
Merged

Top down evaluation custom promise type test#3029
larsewi merged 1 commit into
cfengine:masterfrom
victormlg:top-down-evaluation

Conversation

@victormlg

@victormlg victormlg commented Aug 14, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@nickanderson nickanderson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

A couple small suggestions.

{
inputs => { "../../default.cf.sub" };
bundlesequence => { "test" };
evaluation_order => "top_down";

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🤯

@victormlg victormlg force-pushed the top-down-evaluation branch 2 times, most recently from faf027e to f6fab26 Compare August 18, 2025 12:26
@larsewi

larsewi commented Aug 19, 2025

Copy link
Copy Markdown
Contributor

@victormlg looks like the test failed

./30_custom_promise_types/25_top_down_evaluation_custom_promise_types.cf FAIL (UNEXPECTED FAILURE)

Signed-off-by: Victor Moene <victor.moene@northern.tech>
@victormlg victormlg force-pushed the top-down-evaluation branch from f6fab26 to 6e584a2 Compare August 20, 2025 08:13
@larsewi

larsewi commented Aug 20, 2025

Copy link
Copy Markdown
Contributor

@cf-bottom Jenkins please :)

@cf-bottom

Copy link
Copy Markdown

@larsewi larsewi dismissed nickanderson’s stale review August 21, 2025 08:00

Requested changes have been resolved

@larsewi larsewi merged commit 2655c7b into cfengine:master Aug 21, 2025
31 of 33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants