-
Notifications
You must be signed in to change notification settings - Fork 761
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
Add function use_appveyor() similar to use_travis() #549
Conversation
|
||
#' @rdname infrastructure | ||
#' @export | ||
add_appveyor <- use_appveyor |
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.
Don't need add_appveyor()
existing add functions will eventually be deprecated and removed.
I've rewritten the patch, the refactoring is now gone -- too many special cases. |
#' Add basic AppVeyor template to a package. Also adds \code{appveyor.yml} to | ||
#' \code{.Rbuildignore} so it isn't included in the built package. | ||
#' @export | ||
#' @aliases add_travis |
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.
You can remove that line
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.
Removed. Thanks, good catch.
Added a section to the |
This is great! Hope this makes it to master some day. |
Conflicts: man/infrastructure.Rd
Add function use_appveyor() similar to use_travis()
Thanks! |
AppVeyor is a Windows-based CI system much like Travis CI, and also free for open-source projects. I have prepared a wrapper around r-travis so that AppVeyor can be used to test R packages (and also later deploy them in binary form). It's work in progress, but I think the format of the
appveyor.yml
configuration file is more or less stable.Example: Test output for a build of the
rappdirs
package.Pull request follows.