Skip to content

Conversation

@tonibohnlein
Copy link
Collaborator

No description provided.

Copy link
Collaborator

@raphael-s-steiner raphael-s-steiner left a comment

Choose a reason for hiding this comment

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

Left a couple of comments. Looks good

Copy link
Collaborator Author

@tonibohnlein tonibohnlein left a comment

Choose a reason for hiding this comment

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

looks good

Copy link
Collaborator

@raphael-s-steiner raphael-s-steiner left a comment

Choose a reason for hiding this comment

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

All looks good

Copy link
Collaborator

@papp-pal-andras papp-pal-andras left a comment

Choose a reason for hiding this comment

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

Cool! This must have been fun to do...

(I'll try to do some renaming later when I get the time, e.g. remaining_2 in pebbling, but don't wait for that with the merge)

Copy link
Collaborator

@raphael-s-steiner raphael-s-steiner left a comment

Choose a reason for hiding this comment

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

Looks all good to me now!

@raphael-s-steiner raphael-s-steiner merged commit 18cde89 into master Nov 12, 2025
4 checks passed
@tonibohnlein tonibohnlein deleted the type_fix branch November 12, 2025 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants