Merge pull request #291 from frictionlessdata/remote_zip

Remote zips are now supported

Author: peterdesmet

.github/workflows/R-CMD-check.yaml

@@ -24,7 +24,7 @@ jobs:
           - {os: ubuntu-latest,   r: 'release'}
           - {os: ubuntu-latest,   r: 'oldrel-1'}
           # And minimum supported version in DESCRIPTION
-          - {os: ubuntu-latest, r: '3.6.0'}
+          - {os: ubuntu-latest, r: '4.1.0'}
 
     env:
       GITHUB_PAT: ${{ secrets.GITHUB_TOKEN }}

DESCRIPTION

@@ -32,7 +32,7 @@ URL: https://github.com/frictionlessdata/frictionless-r,
     https://docs.ropensci.org/frictionless/
 BugReports: https://github.com/frictionlessdata/frictionless-r/issues
 Depends:
-    R (>= 3.6.0)
+    R (>= 4.1.0)
 Imports:
     cli,
     dplyr,
@@ -58,4 +58,4 @@ Config/testthat/edition: 3
 Encoding: UTF-8
 LazyData: true
 Roxygen: list(markdown = TRUE)
-RoxygenNote: 7.3.2
+RoxygenNote: 7.3.3

NEWS.md

@@ -2,10 +2,12 @@
 
 * `resources()` is soft-deprecated, please use `resource_names()` instead (#282).
 * `get_schema()` is soft-deprecated, please use `schema()` instead (#282).
+* `read_resource()` now supports reading from remote zip files, thanks to support in {vroom} (1.3.0) (#291).
+* frictionless now relies on R >= 4.1.0 (because of an indirect {vroom} dependency) (#291).
 
 # frictionless 1.2.1
 
-* **frictionless now relies on R version 3.6.0 or higher**. Originally it stated version 3.5.0 or higher, but this was not tested and likely not true (#238).
+* frictionless now relies on R version 3.6.0 or higher. Originally it stated version 3.5.0 or higher, but this was not tested and likely not true (#238).
 * `read_package()` now returns a warning rather than an error when a `datapackage.json` contains no resources. This allows use to create the JSON and then add resources with frictionless (#265).
 * `example_package()` now has a `version` parameter, allowing to load the example Data Package following the Data Package [v1](https://specs.frictionlessdata.io/) or [v2](https://datapackage.org/) specification (#249).
 

man/frictionless-package.Rd

@@ -579,17 +579,7 @@ test_that("read_resource() can read compressed files", {
   )
 
   expect_identical(read_resource(p_local_zip, "deployments"), resource)
-  # Remote zip not supported, see
-  # https://github.com/tidyverse/readr/issues/1042#issuecomment-545103047
-  expect_error(
-    read_resource(p_remote_zip, "deployments"),
-    regexp = paste(
-      "Reading from remote `zip` compressed files is not supported,",
-      "  download the files locally first.",
-      sep = "\n"
-    ),
-    fixed = TRUE
-  )
+  expect_identical(read_resource(p_remote_zip, "deployments"), resource)
   expect_identical(read_resource(p_local_gz, "deployments"), resource)
   expect_identical(read_resource(p_remote_gz, "deployments"), resource)
 })

tests/testthat/test-read_resource.R

@@ -239,4 +239,4 @@ package$resources[[2]]$encoding
 
 [`compression`](https://specs.frictionlessdata.io/patterns/#specification-3) (a recipe) is ignored by `read_resource()` and not set by `add_resource()`.
 
-Compression is derived from the provided `path` instead. If the `path` ends in `.gz`, `.bz2`, `.xz`, or `.zip`, the files are automatically decompressed by `read_resource()` (using default `readr::read_delim()` functionality). Only `.gz` files can be read directly from URL `path`s.
+Compression is derived from the provided `path` instead. If the `path` ends in `.gz`, `.bz2`, `.xz`, or `.zip`, the files are automatically decompressed by `read_resource()` (using default `readr::read_delim()` functionality).