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 automated mode #11

Merged
merged 1 commit into from
May 8, 2020
Merged

Conversation

richm
Copy link
Contributor

@richm richm commented May 6, 2020

My previous commit broke automated mode. This fixes automated mode
and cleans up the code a bit. There are two new cmdline flags:

--workdir /path/to/dir - instead of using the $WORKDIR env. var.,
pass it in with this flag. This will automatically set PRESERVE=true
so as not to accidentally remove any directories under your workdir.

--use-hub - previously it would use hub automatically if found
in the $PATH. Now, you must explicitly set --use-hub in order
to use hub.

My previous commit broke automated mode.  This fixes automated mode
and cleans up the code a bit.  There are two new cmdline flags:

`--workdir /path/to/dir` - instead of using the `$WORKDIR` env. var.,
pass it in with this flag.  This will automatically set `PRESERVE=true`
so as not to accidentally remove any directories under your workdir.

`--use-hub` - previously it would use `hub` automatically if found
in the `$PATH`.  Now, you must explicitly set `--use-hub` in order
to use `hub`.
@richm richm requested review from i386x and pcahyna May 6, 2020 17:09
@richm
Copy link
Contributor Author

richm commented May 6, 2020

This should fix the issues found during the demo, and address the questions @pcahyna asked.

@richm richm merged commit f750c0a into linux-system-roles:master May 8, 2020
@richm richm deleted the fix-automated-mode branch May 8, 2020 14:31
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.

1 participant