Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Version 0.4.0 #9

Merged
merged 15 commits into from
Aug 10, 2024
Merged

Version 0.4.0 #9

merged 15 commits into from
Aug 10, 2024

Conversation

MisterGC
Copy link
Contributor

@MisterGC MisterGC commented Jul 24, 2024

Changes:

  • Support nested invocation of functions (e.g. convert some data and then repeat it)
  • Support referencing a single node when using insert_yaml then only this node gets inserted
  • Use C-based implementation of YAML loader and dumper which is significantly faster
  • Add in-memory cache for loaded YAMLs to improve performance when referencing the same file multiple times. This is especially useful when using the lib variant of zs-yaml in a Python app.
  • Add an example which demonstrates the different aspects of zs-yaml and can be used without copy&paste from the README
  • Reflect Design Principles in the README

Copy link
Contributor

@josephbirkner josephbirkner left a comment

Choose a reason for hiding this comment

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

Please add basic CI which runs the tests. (Edit: I see that there is an issue for that)

@josephbirkner josephbirkner self-requested a review July 31, 2024 11:56
Copy link
Contributor

@josephbirkner josephbirkner left a comment

Choose a reason for hiding this comment

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

Nice 👍

@MisterGC MisterGC merged commit 6465042 into main Aug 10, 2024
1 check passed
@MisterGC MisterGC deleted the 0.4.0 branch August 10, 2024 12:58
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.

2 participants