Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

direnv intigration #10

Merged
merged 5 commits into from
Jun 24, 2024
Merged

direnv intigration #10

merged 5 commits into from
Jun 24, 2024

Conversation

Horiodino
Copy link
Contributor

@Horiodino Horiodino commented Jun 7, 2024

use:
bsf direnv -> initilizes current dir with .envrc and .gitignore
bsf direnv --env key=value -> sets the env in .envrc

by doing bsf direnv direnv files get loaded in the current dir and whenever user opens shell in the current dir ,
it always opens nix dev shell without specifying any argument

[✔️ ] test locally

cmd/init/init.go Outdated Show resolved Hide resolved
cmd/precheck/precheck.go Outdated Show resolved Hide resolved
cmd/precheck/precheck.go Outdated Show resolved Hide resolved
Signed-off-by: Horiodino <[email protected]>
Signed-off-by: Horiodino <[email protected]>
@saiyam1814
Copy link
Contributor

Can you start adding a little description on what this PR aims to solve and tests done for the verification purposes that it works?

cmd/direnv/direnv.go Outdated Show resolved Hide resolved
cmd/direnv/direnv.go Outdated Show resolved Hide resolved
cmd/direnv/direnv.go Outdated Show resolved Hide resolved
Signed-off-by: Horiodino <[email protected]>
@rakshitgondwal
Copy link
Member

I might be missing something over here, but what if a user wants to provide their own .envrc file? How will that work? Should we add a -f flag for such case?

@dr-housemd
Copy link
Contributor

I might be missing something over here, but what if a user wants to provide their own .envrc file? How will that work? Should we add a -f flag for such case?

We just need to add a single line use bsf/. , really. So I'd like us to integrate with it.

cmd/direnv/direnv.go Outdated Show resolved Hide resolved
Signed-off-by: Horiodino <[email protected]>
cmd/direnv/direnv.go Outdated Show resolved Hide resolved
cmd/direnv/direnv.go Outdated Show resolved Hide resolved
cmd/direnv/direnv.go Outdated Show resolved Hide resolved
cmd/direnv/direnv.go Outdated Show resolved Hide resolved
Signed-off-by: Horiodino <[email protected]>
@dr-housemd dr-housemd merged commit 6c4fec9 into main Jun 24, 2024
1 check passed
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.

4 participants