-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Parameterize backup restore script #2412
Parameterize backup restore script #2412
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To keep it backward compatible for anyone who might use this script some another way.
scripts/_lib.sh
Outdated
@@ -60,13 +60,15 @@ function reset() { | |||
} | |||
|
|||
function backup() { | |||
type=$1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
type=$1 | |
type=${1:-"global"} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good suggestion!
It will be of great help to us. I'm waiting for the merge! |
1c7672a
to
c31e5bf
Compare
c31e5bf
to
ee013c9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should update the docs to mention this as well
agreed |
Allows users to call