Skip to content

Simplify Git diff API - #94

Merged
jonasehrlich merged 1 commit into
mainfrom
je/simplify-diff-api
Aug 5, 2025
Merged

Simplify Git diff API#94
jonasehrlich merged 1 commit into
mainfrom
je/simplify-diff-api

Conversation

@jonasehrlich

Copy link
Copy Markdown
Owner

The API now only provides the following data:

struct Diff {
    /// Patch between old and new
    patch: String,
    /// Stats of the diff
    stats: DiffStats,
    /// Map of old source paths to the old content
    old_sources: hash_map::HashMap<Path, FileContent>,
}

This is easier to handle on the client side.

@jonasehrlich jonasehrlich self-assigned this Aug 3, 2025
@jonasehrlich jonasehrlich added frontend This affects the frontend backend This affects the backend labels Aug 3, 2025

@jonasehrlich jonasehrlich left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

⚠️ Requires #93 to be merged

@TobTheRock

TobTheRock commented Aug 5, 2025

Copy link
Copy Markdown
Collaborator

Github really lacks merge dependencies, some comments on #95 apply actually to this change..

@jonasehrlich
jonasehrlich force-pushed the je/simplify-diff-api branch from e4b13c4 to b0fba1b Compare August 5, 2025 18:45
@jonasehrlich
jonasehrlich force-pushed the je/simplify-diff-api branch from b0fba1b to 0ad24d6 Compare August 5, 2025 18:46
@jonasehrlich
jonasehrlich merged commit ce7a5f9 into main Aug 5, 2025
10 checks passed
@jonasehrlich
jonasehrlich deleted the je/simplify-diff-api branch August 5, 2025 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend This affects the backend frontend This affects the frontend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants