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.

Sign up for GitHub

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

Jump to bottom

Display description of exported environment variables along with aliases and functions #56

Closed
rafaelbco opened this issue Feb 16, 2016 · 1 comment
Closed

Display description of exported environment variables along with aliases and functions #56

rafaelbco opened this issue Feb 16, 2016 · 1 comment

Comments

@rafaelbco
Copy link

I think it would be useful if I could do something like that in a desk file:

# Description: my desk to work on things.

# Do something.
alias mycommand="foo"

# My environment variable.
export MYVAR="something"

And then when typing desk with the desk activated it would show the description of $MYVAR:

mydesk - my desk to work on things.

  mycommand  Do something.

Environment variables:

$MYVAR  My environment variable.
@cfont
Copy link
Contributor

cfont commented Mar 8, 2016

@jamesob, would you like me to create a PR related to this so you can check it out and determine if it indeed fixes this issue the way you would want?

@cfont cfont mentioned this issue Mar 10, 2016
Merged
cfont added a commit to cfont/desk that referenced this issue Mar 15, 2016
@cfont
Adding test scenarios for displaying exported variables
0e364e8 
This commit includes two file updates for testing the new feature
as requested in jamesob#56. The example hello deskfile now has
a variable set called MyName with a value "James" and the runtests
shell script includes a test to echo the value of that variable.

This test proves that the variable is exported as expected from the
deskfile and is retrievable as expected.

Still #todo includes a test that lists the description of the deskfile
and looks for that variable to be listed along with the aliases and
functions.
jamesob added a commit that referenced this issue Mar 19, 2016
@jamesob
Merge pull request #60 from cfont/features/display_exports
4cab6dd 
Fixes #56 adding support for displaying exported environment variables in deskfiles
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@cfont @rafaelbco