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

rc: add DIRENV_STATUS with status of direnv loading #1010

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

Conversation

vincentbernat
Copy link
Contributor

It can be "denied" if the environment loading was denied. "failed" if the environment failed to load. "allowed" if everything was successful.

Copy link
Member

@zimbatm zimbatm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you also add it to the direnv status command output? Otherwise looks good.

@vincentbernat
Copy link
Contributor Author

I am not sure what to put exactly. There is already "RC allowed true/false" in the output.

@branchvincent branchvincent mentioned this pull request Apr 2, 2023
4 tasks
@branchvincent
Copy link
Contributor

This would be great to have! That way other tools (like terminal prompts) can check the status without calling direnv

@branchvincent
Copy link
Contributor

Is there anything I can do to help push this forward?

@vincentbernat
Copy link
Contributor Author

From my side, I don't know what to do as there is already something in direnv status to indicate if the environment is loaded or not, unless I am misunderstanding the request.

It can be "denied" if the environment loading was denied. "failed" if
the environment failed to load. "allowed" if everything was successful.
When nothing is imported, it is considered a failure.
@vincentbernat
Copy link
Contributor Author

I have rebased the patch to ensure it still works with b80f567.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants