Skip to content

Fix backward compatibility with TOML format produced by insta < 1.45.0#849

Merged
max-sixty merged 2 commits intomitsuhiko:masterfrom
chitoku-k:fix/toml
Dec 29, 2025
Merged

Fix backward compatibility with TOML format produced by insta < 1.45.0#849
max-sixty merged 2 commits intomitsuhiko:masterfrom
chitoku-k:fix/toml

Conversation

@chitoku-k
Copy link
Contributor

@chitoku-k chitoku-k commented Dec 29, 2025

This PR fixes backward compatibility with insta < 1.45.0 that has been broken since #834.

As far as I tested, the tests that were added in #834 did not match the behavior before insta 1.45.0 and simply produced new behavior. I tried to fix the implementation as long as test cases and ensured it locally by adding back toml 0.5.7 and temporarily reverting back to toml::to_string_pretty(&content) to test if it matches the original behavior except for those that panicked before, which does not break compatibility at all.

@max-sixty
Copy link
Collaborator

thank you very much @chitoku-k! I think you're correct; apologies for releasing the backward-incompat code...

@max-sixty max-sixty merged commit 00b1cd4 into mitsuhiko:master Dec 29, 2025
15 checks passed
@max-sixty max-sixty mentioned this pull request Dec 29, 2025
max-sixty added a commit that referenced this pull request Dec 29, 2025
## Summary

- Bumps version to 1.45.1
- Updates CHANGELOG with the fix from #849

## Changes

- Fix backward compatibility with TOML format produced by insta <
1.45.0. #849

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude <noreply@anthropic.com>
@chitoku-k chitoku-k deleted the fix/toml branch December 29, 2025 03:02
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