Skip to content

Add load command#80

Open
ryansch wants to merge 6 commits into
jamesob:masterfrom
outstand:load
Open

Add load command#80
ryansch wants to merge 6 commits into
jamesob:masterfrom
outstand:load

Conversation

@ryansch

@ryansch ryansch commented Mar 20, 2018

Copy link
Copy Markdown

Adds the ability to load a desk into the current shell. Useful for things like tmux where we've already started a new shell.

Usage: eval $(desk load <name|dir>)

Fixes #36

@megalithic

Copy link
Copy Markdown

@jamesob bump, pls

@megalithic

Copy link
Copy Markdown

@jamesob bump, pls; this is brilliant.

@jamesob

jamesob commented Jul 6, 2022

Copy link
Copy Markdown
Owner

Hey, sorry for the egregious delay here and thank you for the reminder. Looks good, thanks!

@jamesob

jamesob commented Jul 6, 2022

Copy link
Copy Markdown
Owner

The included test fails for me when I run it locally with make bash:

desktester@1b19b608cca5:~$ bash run_tests.sh
Desk dir already exists at /home/desktester/.desk
mkdir: cannot create directory '/home/desktester/terraform-repo': File exists
~/example-project ~
~
~/example-project ~
/bin/bash: line 0: export: `${DESK_ENV}': not a valid identifier
grep: "/home/desktester/example-project/Deskfile";: No such file or directory
grep: "/home/desktester/example-project/Deskfile";: No such file or directory
Failed: Deskfile load didn't work (./)

@ryansch

ryansch commented Jul 7, 2022

Copy link
Copy Markdown
Author

@jamesob Try now!

@ryansch

ryansch commented Jul 7, 2022

Copy link
Copy Markdown
Author

I also tossed in some Dockerfile updates.

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.

Load a desk without launching a new shell instance

3 participants