-
Notifications
You must be signed in to change notification settings - Fork 3
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
Accompanying change for s3mpi change to support safe = TRUE/FALSE #26
Conversation
# Since running syberia files is a reproducible process there | ||
# shouldn't be a lot of damage done | ||
# There is an option to overwrite this behavior | ||
if (is.element('safe', names(opts))) args$safe <- opts$safe |
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.
this is the non-trailing-whitespace change
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.
Why not
args$safe <- opts$safe %||% args$safe
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.
Just to have the same style in two adjacent lines
if (is.element('safe', names(opts))) args$safe <- opts$safe
if (is.element('s3path', names(opts))) args$.path <- opts$s3path
Should I refactor both before merge?
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.
Yes please, I don't know what I was thinking.
@@ -1,7 +1,7 @@ | |||
Package: syberiaStages | |||
Type: Package | |||
Title: Syberia Stages | |||
Version: 0.2.1 | |||
Version: 0.3.0 |
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.
0.2.2 please
Looks good! |
Accompanying change for s3mpi change to support safe = TRUE/FALSE
Bumped the package version so that we can later enforce it using the lockfile