Skip to content

Update reference to upstream direnv-hook.sh file#7

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-6
Draft

Update reference to upstream direnv-hook.sh file#7
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-6

Conversation

Copy link
Copy Markdown

Copilot AI commented Sep 7, 2025

This PR adapts to the breaking change introduced in numtide/prj-spec#8, where the contrib/direnv file has been renamed to contrib/direnv-hook.sh.

The upstream change refactored the prj-spec direnv integration by:

  • Renaming contrib/direnv to contrib/shell-hook.sh (generic shell hook)
  • Creating a new contrib/direnv-hook.sh that sources the shell hook and adds direnv-specific functionality

This maintains the same functionality for direnv users while providing a cleaner separation between generic shell integration and direnv-specific features.

Changes Made

  • Updated the fetchurl URL from contrib/direnv to contrib/direnv-hook.sh
  • Updated the commit SHA to reference the PR branch: 4727ff5b89020912ead606c9ebbbe3e94d4a9f50
  • Updated the SHA256 hash to match the new file content: sha256-iTOGxlNyuWM5w581kGpzTU1joX1UUouNaQKQ63LM+6Y=

The changes are minimal (only 2 lines) and maintain full backward compatibility for existing users. All functionality remains the same.

Fixes #6.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copy link
Copy Markdown
Author

Copilot AI commented Sep 7, 2025

@blaggacao 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs.

I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review.

Co-authored-by: blaggacao <7548295+blaggacao@users.noreply.github.com>
Copilot AI changed the title [WIP] Change in upstream file name Update reference to upstream direnv-hook.sh file Sep 7, 2025
Copilot AI requested a review from blaggacao September 7, 2025 17:45
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.

Change in upstream file name

2 participants