Skip to content

vendor imara-diff#2506

Draft
Sebastian Thiel (Byron) wants to merge 4 commits intomainfrom
vendor-imara-diff
Draft

vendor imara-diff#2506
Sebastian Thiel (Byron) wants to merge 4 commits intomainfrom
vendor-imara-diff

Conversation

@Byron
Copy link
Copy Markdown
Member

@Byron Sebastian Thiel (Byron) commented Apr 7, 2026

Take imara-diff in and unify both versions of it, one that has sliders, the other one that doesn't.
See what LLMs can do as well if they see both versions at the same time easily.

Tasks

  • first vendor with separate crates for 0.1 and 0.2, used by all code and it compiles
  • refackiew fix-ci
  • cleanup
    • merge everything into one crate, reuse shared types, etc.

Copy link
Copy Markdown

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

zizmor found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

…doring

This is the baseline version, with a DoS issue fixed due to out-of-bound
computation.

Drop large JSON performance-test fixtures and the perf-only test that used them.
…doring

It contains a DoS fix for a runaway computation.

Drop large JSON performance-test fixtures and the perf-only test that used them.
@Byron Sebastian Thiel (Byron) force-pushed the vendor-imara-diff branch 2 times, most recently from 10f4aa6 to 96ee65b Compare April 10, 2026 02:49
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