Skip to content

fix: don't make perSystemFwd a function#370

Merged
vic merged 2 commits intodenful:mainfrom
theutz:patch-3
Apr 3, 2026
Merged

fix: don't make perSystemFwd a function#370
vic merged 2 commits intodenful:mainfrom
theutz:patch-3

Conversation

@theutz
Copy link
Copy Markdown
Collaborator

@theutz theutz commented Apr 3, 2026

Fixes #367

Fixes denful#367 

Signed-off-by: Michael Utz <theutz@users.noreply.github.com>
@theutz
Copy link
Copy Markdown
Collaborator Author

theutz commented Apr 3, 2026

Fixes #371

@vic
Copy link
Copy Markdown
Member

vic commented Apr 3, 2026

I guess we need a an aditional flake check at that template because that bug was not catched by any of the CI checks. Feel like adding some check that help us prevent it again ?

EDIT: I will merge, but if you want to send another PR with a flake check for this feel free.

@vic vic added the allow-ci allow all CI integration tests label Apr 3, 2026
@vic vic merged commit b258d65 into denful:main Apr 3, 2026
24 of 25 checks passed
@theutz
Copy link
Copy Markdown
Collaborator Author

theutz commented Apr 3, 2026

Will do! Interestingly, the checks passed... but only because they effectively didn't run. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

allow-ci allow all CI integration tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants