Skip to content

UPSTREAM PR #2532: Use GIX_TEST_FIXTURE_HASH for gix-diff#49

Open
loci-dev wants to merge 8 commits into
mainfrom
loci/pr-2532-run-gix-diff-tests-with-sha-256
Open

UPSTREAM PR #2532: Use GIX_TEST_FIXTURE_HASH for gix-diff#49
loci-dev wants to merge 8 commits into
mainfrom
loci/pr-2532-run-gix-diff-tests-with-sha-256

Conversation

@loci-dev
Copy link
Copy Markdown

Note

Source pull request: GitoxideLabs/gitoxide#2532

This is a follow-up to #2497 which made tests in gix-diff-tests run with GIX_TEST_FIXTURE_HASH=sha256.

This PR adds:

  • Remove profile.release section in gix-imara-diff
    This fixes the following warning:

    warning: profiles for the non root package will be ignored, specify profiles at the workspace root:
    package:   .../gix-imara-diff/Cargo.toml
    workspace: .../Cargo.toml
    

    There’s also a commented out [[bench]] section in gix-imara-diff/Cargo.toml that could be removed as well.

  • Run gix-diff-tests with GIX_TEST_FIXTURE_HASH=sha1 as well (before, they were only run with GIX_TEST_FIXTURE_HASH=sha256)

  • Use GIX_TEST_FIXTURE_HASH for gix-diff as well (as opposed to gix-diff-tests which is a separate crate)

This fixes the following warning:

```
warning: profiles for the non root package will be ignored, specify profiles at the workspace root:
package:   .../gix-imara-diff/Cargo.toml
workspace: .../Cargo.toml
```
Collapse match as the two branches are not identical as they should have
been from the start.
@loci-review
Copy link
Copy Markdown

loci-review Bot commented Apr 25, 2026

The analysis encountered an error. Please review the Processing Details for more information.

1 similar comment
@loci-review
Copy link
Copy Markdown

loci-review Bot commented Apr 25, 2026

The analysis encountered an error. Please review the Processing Details for more information.

@loci-dev loci-dev force-pushed the main branch 3 times, most recently from c6739bc to 6f67b12 Compare April 29, 2026 07:20
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