Merge pull request #227 from elipousson/fix-dots-error-message

Improve `arc_select()` error message per #226

Author: JosiahParry

Diff for: NEWS.md

@@ -4,6 +4,10 @@
 
 - `dplyr` methods for `collect()`, `select()`, and `filter()` have been removed. <https://github.com/R-ArcGIS/arcgislayers/issues/111> <https://github.com/R-ArcGIS/arcgislayers/issues/224> <https://github.com/R-ArcGIS/arcgislayers/issues/218>
 
+## Bug fixes
+
+- `arc_select()` includes argument name in error message when `...` contains non-string values. <https://github.com/R-ArcGIS/arcgislayers/issues/226>
+
 # arcgislayers 0.3.1
 
 ## Bug fixes

Diff for: R/arc-select.R

@@ -101,7 +101,7 @@ arc_select <- function(
     key <- dots_names[i]
     val <- dots[[i]]
     # check that the value is a scalar and non-empty
-    check_string(val, allow_empty = FALSE)
+    check_string(val, arg = key, allow_empty = FALSE)
 
     # insert into query
     query[[key]] <- val