Skip to content

Add Copilot instructions to ignore sha256 comment mismatches in cabal.project#1145

Open
Copilot wants to merge 1 commit intomasterfrom
copilot/sub-pr-1144
Open

Add Copilot instructions to ignore sha256 comment mismatches in cabal.project#1145
Copilot wants to merge 1 commit intomasterfrom
copilot/sub-pr-1144

Conversation

Copy link
Contributor

Copilot AI commented Mar 19, 2026

Changelog

- description: |
    Add Copilot instructions to ignore sha256 comment mismatches in cabal.project
  type:
  - maintenance
  projects:
  - cardano-api

Context

The automated Copilot reviewer was incorrectly flagging --sha256: comment lines in cabal.project SRP stanzas as inconsistent with flake.nix sha256 values. These hashes are intentionally different — they are computed using different fetch methods (Nix tarball fetch vs. fetchgit).

Adds a new section to .github/copilot-instructions.md documenting that --sha256: comments in cabal.project SRP stanzas are consumed by Nix tooling (not cabal), and that their values will intentionally differ from flake.nix sha256s. Instructs Copilot not to flag this as an error.

How to trust this PR

Documentation-only change to .github/copilot-instructions.md. No code changes.

Checklist

  • Commit sequence broadly makes sense and commits have useful messages
  • New tests are added if needed and existing tests are updated. See Running tests for more details
  • Self-reviewed the diff

Copilot AI changed the title [WIP] [WIP] Address feedback on switching proto-lens SRP to google upstream master Add Copilot instructions to ignore sha256 comment mismatches in cabal.project Mar 19, 2026
Copilot AI requested a review from carbolymer March 19, 2026 12:23
@carbolymer carbolymer marked this pull request as ready for review March 19, 2026 14:17
@carbolymer carbolymer requested review from a team, Jimbo4350, newhoggy and palas as code owners March 19, 2026 14:17
@carbolymer carbolymer force-pushed the copilot/sub-pr-1144 branch from cfc032f to 7e9f1ec Compare March 19, 2026 14:25
@carbolymer carbolymer changed the base branch from mgalazyn/chore/update-proto-lens-srp to master March 19, 2026 14:28
@carbolymer carbolymer enabled auto-merge March 19, 2026 14:28
@carbolymer carbolymer force-pushed the copilot/sub-pr-1144 branch from 7e9f1ec to 33459e6 Compare March 20, 2026 11:51
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