Thanks to visit codestin.com
Credit goes to github.com

Skip to content

chore(localdev): add path management to .envrc [skip ci]#7516

Merged
rfay merged 2 commits intoddev:mainfrom
rfay:20250807_rfay_direnv
Aug 8, 2025
Merged

chore(localdev): add path management to .envrc [skip ci]#7516
rfay merged 2 commits intoddev:mainfrom
rfay:20250807_rfay_direnv

Conversation

@rfay
Copy link
Member

@rfay rfay commented Aug 7, 2025

The Issue

  • Add PATH management to direnv so it can automatically pick up the ddev built in the current directory
  • Minor tweak to .gitignore to make it pick up a bit more of .claude

Manual Testing Instructions

direnv allow

which ddev should point to the ddev built in the current environment

Note

direnv support with .envrc was added in

@rfay rfay marked this pull request as ready for review August 7, 2025 20:20
@rfay rfay requested a review from a team as a code owner August 7, 2025 20:20
@rfay rfay force-pushed the 20250807_rfay_direnv branch from d9e1cc6 to d87351f Compare August 7, 2025 20:21
@rfay rfay requested a review from stasadev August 7, 2025 20:24
@stasadev stasadev force-pushed the 20250807_rfay_direnv branch from d87351f to 7487318 Compare August 8, 2025 11:12
Copy link
Member

@stasadev stasadev left a comment

Choose a reason for hiding this comment

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

Looks good to me!

I made a commit 7487318 to simplify the logic.


BIN_DIR="$PWD/.gotmp/bin/${OS}_${ARCH}"

BIN_DIR="$PWD/.gotmp/bin/$(go env GOHOSTOS)_$(go env GOHOSTARCH)"
Copy link
Member Author

Choose a reason for hiding this comment

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

This is better, thanks!

@rfay
Copy link
Member Author

rfay commented Aug 8, 2025

Thanks, we'll see how this works out for us. I won't be surprised if we have to adjust it.

BTW there is a plugin for jetbrains tools to make it do envrc, you probably already have it. and VS Code has the feature by default I think.

@rfay rfay merged commit e9605db into ddev:main Aug 8, 2025
4 checks passed
@rfay rfay deleted the 20250807_rfay_direnv branch August 8, 2025 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments