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

Insta 2 #466

Closed
wants to merge 5 commits into from
Closed

Insta 2 #466

wants to merge 5 commits into from

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
Sponsor Contributor

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.
2 participants