Skip to content

Conversation

@TobiasNx
Copy link
Contributor

@TobiasNx TobiasNx commented Jul 31, 2025

Related to metafacture/metafacture-core#704

These should be the new and adjusted examples in the playground for the set_ functions.

@blackwinter do you think the set_ function examples are good enough?

@blackwinter
Copy link
Member

You should reset your branch. It also includes the changes from #226.

Copy link
Member

@blackwinter blackwinter left a comment

Choose a reason for hiding this comment

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

Apart from the branch mishap, 👍.

@TobiasNx
Copy link
Contributor Author

@blackwinter I have reset the branch to exclude the àdd_function examples. But now theset_andadd_` functions should be matching.

@TobiasNx TobiasNx requested a review from blackwinter December 16, 2025 19:43
@TobiasNx TobiasNx assigned blackwinter and unassigned TobiasNx Dec 16, 2025
Copy link
Member

@blackwinter blackwinter left a comment

Choose a reason for hiding this comment

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

Inconsistent wording "is missing" vs. "is not provided", but negligible.

@blackwinter blackwinter assigned TobiasNx and unassigned blackwinter Dec 17, 2025
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.

Add fix examples for new functions

3 participants