-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
[R-package] [ci] Add Windows CI for R package (fixes #2335) #2936
Merged
Merged
Changes from 1 commit
Commits
Show all changes
93 commits
Select commit
Hold shift + click to select a range
19ed854
first test of appveyor
jameslamb 16476da
strings are strings
jameslamb f7573f0
lil bit of build script
jameslamb 311d3d8
fixing paths
jameslamb c422b15
removed unnecessary file
jameslamb b2fce60
updated CRAN URL
jameslamb b973b7d
added a lot more printing
jameslamb a8873f6
fixing paths
jameslamb 952ca55
more stuff
jameslamb b4cf398
fixed paths
jameslamb cc6d02b
more stuff
jameslamb adfb832
more path guessing
jameslamb 92b0032
even more paths
jameslamb 8d77941
more stuff
jameslamb b7e2abe
moar logz
jameslamb 12fb638
and now for something totally different
jameslamb 1091504
please work
jameslamb 96dcc18
ok could be a thing
jameslamb 97d9913
changing directories
jameslamb 6ed498d
we might be in business
jameslamb b275d22
fixed install syntax
jameslamb f701325
tryinv mingw
jameslamb 618086a
more mingw
jameslamb 90f91e5
ignore Suggests check
jameslamb 2f4979a
trying Azure DevOps
jameslamb 3b07c7b
just run bare minimum for Azure DevOps
jameslamb 2d17cae
fixed build dir thing
jameslamb 2e9fe09
trying to set libPaths
jameslamb 855cdc9
more testing
jameslamb 6d8878b
trying R 3.6.3
jameslamb c854fe3
R 3.6.3
jameslamb 86f4eec
this feels right
jameslamb 8b12858
still messing around with libraries
jameslamb dc50ad5
more paths
jameslamb 065df95
removed duplication in Windows testing code
jameslamb 432df25
simpler
jameslamb 6e73047
fixed conda stuff
jameslamb fec3de7
more conda stuff
jameslamb 9b90d84
more fixes
jameslamb c305e9d
fixed testing script
jameslamb c70e352
moved AppVeyor setup to the top
jameslamb 517e519
commenting
jameslamb 56bf28a
ch-ch-ch-ch-chaaaanges
jameslamb f439ba2
paths
jameslamb 2923111
plz work
jameslamb 65915a1
fixed conda stuff in Windows CI
jameslamb f18399d
uncommented stuff to test a full build
jameslamb 1de8f6c
fixed quotes and removed some unnecessary stuff
jameslamb a912213
added install.libs.R change
jameslamb 290e5b7
quotes are impoortant
jameslamb 2565df7
added commented-out stuff back in
jameslamb cd89065
added Windows script, download retries, and MSVC linking
jameslamb fbf494c
minor fixes
jameslamb 244a14a
cleaned up debugging code in FindLibR
jameslamb 42609cd
cleaned up debugging code and moved R first in CI
jameslamb 2804d45
fixed vsts-ci indentation
jameslamb af3c004
cut documentation stuff out of MSVC build
jameslamb 1c498d0
fix R CMD check for Azure
jameslamb 9e6aa49
misc whitespace changes
jameslamb 76d264d
merge in #2962
jameslamb 01359ab
Merge branch 'master' into ci/r-windows-ci
jameslamb d100027
Added echoing of build logs from R CMD check
jameslamb 912a2fd
cut out more documentation tests
jameslamb d76b551
fixed NOTE about imports from Matrix
jameslamb 7cdfc62
Merge branch 'master' into ci/r-windows-ci
jameslamb 1a2053f
Merge branch 'master' into ci/r-windows-ci
jameslamb bdf10e0
Merge branch 'master' into ci/r-windows-ci
jameslamb 88a7a09
moved some changes out of this PR and into #2963
jameslamb ae813f2
fixed whitespace stuff
jameslamb 1b2363a
added check on number of NOTES
jameslamb dd52dbf
adding better checks
jameslamb 7f9eb7c
fixing check on NOTEs
jameslamb 374ba31
removing unnecessary variable
jameslamb 30d8af1
Update .ci/test_r_package_windows.ps1
jameslamb e3b9ac8
some changes
jameslamb 419e39b
fix quoting
jameslamb bfebf1d
trying MINGW on Azure DevOps
jameslamb 3e2dcfd
fixing paths
jameslamb 8ba021a
more paths
jameslamb 51e4d2d
fixing paths
jameslamb 5d74019
testing paths
jameslamb d7c160b
fixing slashes
jameslamb a8582db
pinned CTAN mirror
jameslamb 8264f4a
get better logs
jameslamb d843795
made sure Azure finds MinGW, fixed search for LIBR_CORE_LIBRARY, stop…
jameslamb 10bd18d
Apply suggestions from code review
jameslamb 6ca2be8
added CXX, CC for Windows builds and changed back to building docs on…
jameslamb cdf1df8
stored LIBR_CORE_LIBRARY hints in one variable
jameslamb 9c59f33
Apply suggestions from code review
jameslamb 747378e
changes from code review
jameslamb c0b87d6
increased parallel builds for Azure CI
jameslamb 6820fc6
Apply suggestions from code review
jameslamb 7075b80
Merge branch 'master' into ci/r-windows-ci
jameslamb File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
fixing paths
- Loading branch information
commit 3e2dcfd92fe235b0c2b03955ff51fa0c4ac2de1a
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Do you think we should test with R 4.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.
yes I think we should have a separate issue for testing on R 4.0.0 across all platforms, now that it's officially released. Good call!
I just opened #3024 to capture that work (and added it to #2302 )