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

Add from env #808

Merged
merged 2 commits into from
Apr 3, 2024
Merged

Add from env #808

merged 2 commits into from
Apr 3, 2024

Conversation

cptpiepmatz
Copy link
Contributor

This adds a little command that allows opening .env files as records.
My implementation removes comments and works with both quoted and unquoted .env files.

modules/formats/from-env.nu Outdated Show resolved Hide resolved
modules/formats/from-env.nu Outdated Show resolved Hide resolved
@fdncred
Copy link
Collaborator

fdncred commented Apr 2, 2024

Just a couple comments, neither is a deal breaker, just tips.

@cptpiepmatz cptpiepmatz requested a review from fdncred April 3, 2024 09:09
@fdncred fdncred merged commit df90d65 into nushell:main Apr 3, 2024
1 check passed
@fdncred
Copy link
Collaborator

fdncred commented Apr 3, 2024

thanks

csc530 pushed a commit to csc530/nu_scripts that referenced this pull request Apr 7, 2024
This adds a little command that allows opening .env files as records.
My implementation removes comments and works with both quoted and
unquoted .env files.

add windows compatibility for command not found hook (nushell#811)

the hook (did_you_mean.nu) had stopped working for me recently, I added
an if branch for Windows to use their `Path` env var (not `PATH`) and
only list files that match in the `PATHEXT` env var to resolve my issue
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.

None yet

2 participants