Skip to content

Conversation

@scarlehoff
Copy link
Member

@scarlehoff scarlehoff commented Oct 28, 2025

Instead of waiting for hours for the EKO to be produced only to find out at the end that you couldn't use it.

@felixhekhorn felixhekhorn added the refactor Refactor code label Oct 29, 2025
@scarlehoff scarlehoff force-pushed the move_the_check_to_the_op branch from 4191822 to 4375fad Compare November 4, 2025 10:59
scarlehoff and others added 2 commits November 4, 2025 11:59
Co-authored-by: Felix Hekhorn <felixhekhorn@users.noreply.github.com>
@scarlehoff scarlehoff force-pushed the move_the_check_to_the_op branch from 4375fad to 0d12492 Compare November 4, 2025 10:59
@scarlehoff
Copy link
Member Author

@evagroenendijk I needed to remove the delete to be able to combine different fktables. Not sure whether we just forgot to remove it somehow from your branch or something? (I had rebased this on top of main so it should include your changes)

@scarlehoff
Copy link
Member Author

@evagroenendijk can I merge this? (i.e., is the removal of the delete ok?)

@scarlehoff scarlehoff merged commit f9fcd2c into main Nov 11, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Refactor code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants