Skip to content

Conversation

@mitsuhiko
Copy link
Owner

@mitsuhiko mitsuhiko commented Mar 26, 2024

This is a pull request that accumulates all breaking changes for insta 2.

Fixes #439
Fixes #446
Refs #372
Refs #461

@max-sixty
Copy link
Collaborator

Would you be up for this change too @mitsuhiko ? #456 (comment)

(some bigger ideas at the top, the link is to the narrower --- change in the second comment)

max-sixty added a commit to max-sixty/insta that referenced this pull request Mar 26, 2024
This is the code for mitsuhiko#456 (comment), as mentioned in mitsuhiko#466.

There's a very small change in yaml inline snapshots — shown here in the tests. In return, it makes the macros simpler & more maintainable.
@mitsuhiko
Copy link
Owner Author

Closing this for now. Will approach this from scratch.

@mitsuhiko mitsuhiko closed this May 15, 2024
@mitsuhiko mitsuhiko deleted the insta-2 branch May 15, 2024 09:38
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.

assert_toml_snapshot produces invalid TOML when the toml library would generate an error.

3 participants