Skip to content

Conversation

@Plamper
Copy link
Collaborator

@Plamper Plamper commented Oct 3, 2025

I am too lazy to always run nix develop.

@Plamper Plamper enabled auto-merge (squash) October 3, 2025 06:29
@drafolin
Copy link
Collaborator

drafolin commented Oct 3, 2025

@nyabinary does this make sense from a nix point of view?

@Plamper
Copy link
Collaborator Author

Plamper commented Oct 3, 2025

The gitignore is just build dirs. The .envrc only does something if direnv and nix direnv are installed and has to be manually approved. This is basically a convenience thing.

@drafolin
Copy link
Collaborator

drafolin commented Oct 3, 2025

The gitignore is just build dirs. The .envrc only does something if direnv and nix direnv are installed and has to be manually approved. This is basically a convenience thing.

I'm mostly wondering if it makes sense to merge pr's whose whole description is, "I'm to lazy to [...]", and if it might clutter the repository with single dev-specific files

@Plamper
Copy link
Collaborator Author

Plamper commented Oct 3, 2025

I see your point. At least the .gitignore changes should be merged imo, as building the nix project using nix build will create that result folder. But I think it's relatively common to have the .envrc as well because this will ensure the code editor is also inside the nix env.

@drafolin drafolin removed their request for review October 15, 2025 06: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