-
Notifications
You must be signed in to change notification settings - Fork 0
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
Export windows functions directly from hermod #18
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #18 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 8 9 +1
Lines 354 357 +3
=========================================
+ Hits 354 357 +3 ☔ View full report in Codecov by Sentry. |
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.
Looks good - one drive-by edit on the docs
##' @param drive_remote The place to mount the drive on the cluster. | ||
##' We're probably going to mount things at Q: and T: already so | ||
##' don't use those. And things like C: are likely to be used. | ||
##' Perhaps there are some guidelines for this somewhere? |
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.
Future to-dos -
- Should we have a ticket to write this guidance and improve that last sentence!
- Should we outlaw
I:
here? - I slightly feel we should drift away from the temp drive, but perhaps not quite ready to shout that yet.
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.
- https://mrc-ide.myjetbrains.com/youtrack/issue/mrc-4790 - added a ticket for
I:
- https://mrc-ide.myjetbrains.com/youtrack/issue/mrc-4791 - for docs
For the temp drive we can probably just softly move away from it?
"only need to call 'library(hermod)'") | ||
packageStartupMessage(paste(strwrap(msg), collapse = "\n")) | ||
# nocov end | ||
} |
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.
Nice!
Co-authored-by: Wes Hinsley <[email protected]>
This package prevents users needing to interact at all with hermod.windows - hermod becomes the wrapper for the call