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

Silent/Quiet option when opening a new terminal #435

Closed
jgoux opened this issue Feb 24, 2023 · 2 comments · Fixed by #1379
Closed

Silent/Quiet option when opening a new terminal #435

jgoux opened this issue Feb 24, 2023 · 2 comments · Fixed by #1379
Labels
enhancement New feature or request

Comments

@jgoux
Copy link

jgoux commented Feb 24, 2023

Hello 👋, very excited about this project!

Could we add an ENV variable or an option to suppress the Building shell ... and other progress reports when opening a new terminal? Ideally, I'd like to only see errors reported.

The related code seems here:

devenv/src/devenv.nix

Lines 59 to 65 in 1aa3dbb

function shell {
assemble
echo "Building shell ..." 1>&2
env=$($CUSTOM_NIX/bin/nix $NIX_FLAGS print-dev-env --impure --profile "$DEVENV_GC/shell")
$CUSTOM_NIX/bin/nix-env -p "$DEVENV_GC/shell" --delete-generations old 2>/dev/null
ln -sf $(${pkgs.coreutils}/bin/readlink -f "$DEVENV_GC/shell") "$GC_DIR-shell"
}

@domenkozar
Copy link
Member

Sounds good! I'd like to refactor the argument parsing by using docopts, so things like this will be possible.

@domenkozar domenkozar added the enhancement New feature or request label Feb 27, 2023
@alicebob
Copy link

alicebob commented Mar 7, 2023

This would also be nice for github actions. My actions have multiple devenv shell statements, which don't need the devenv output:

    - name: Test formatting
      run: devenv shell test -z '$(go fmt ./...)'
    - name: Run tests
      run: devenv shell make citest

(-quiet or similar would work great for my use case)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants