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

fix: remove try block #949

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix: remove try block #949

wants to merge 1 commit into from

Commits on May 25, 2022

  1. fix: remove try block

    `try {} except e {}` is now deprecated (elvish v0.18.0) and will be
    removed in the next version - 0.19.0.
    
    Since direnv will most probably run in pre-0.18.0 elvish
    environments it is maybe for the best to remove the try block.
    
    As far as I can see - its use is to catch exceptions from running
    `direnv export elvish`, which should only happen if direnv vanishes
    from the user's path. Any other error should most probably fail
    anyway, rather than echo the exception but proceed.
    avioli committed May 25, 2022
    Configuration menu
    Copy the full SHA
    13ef247 View commit details
    Browse the repository at this point in the history