Closed KevCaz closed 3 years ago
Some issue on window for a specific test, I'm investigating (kind of harder without a Windows machine though),
I had a lot of trouble with windows, turn out part of the problem was with the virtual environment.... see https://github.com/actions/virtual-environments/issues/712, the second solution solves this!
@KevCaz To avoid all these set-env
warnings you can use the new syntax echo "TEMP=$env:USERPROFILE\AppData\Local\Temp" >> $env:GITHUB_ENV
Yep, that's what I ended up doing!
This looks absolutely awesome. Will review this on Monday.
Also, I agree about what you said for downloads and html files.
Hi @VLucet ,
I took some time to rewrite
govcan_dl_resources()
, in a nutshell :ckan_package_stack
,ckan_package
,ckan_resource
,ckan_resource_stack
andcharacter
.ìd_as_filename
) to use resource id as file name, this was needed as sometimes different resources have the same names (...:):).tibble
) is returned to identify which files were downloaded (see example below indetails
) and to gather all URLs.Note that documents such as html or wms are not downloaded (and for the moment I honestly think it is for the best). Also, because of the difficulties with "session" in
ckan_fetch()
, I think it makes more sense to just store files locally, so I always usestore = "disk"
but this cam always be changed in a future release.Let me know what do you think but I think with a little more work on this, you will be closed to a nice first release to the CRAN. I can (of course) give you a hand for the doc and to add more tests.