[LeonidasZhak]

Summary

When col_select results in all columns being skipped (e.g. via any_of() with non-existent columns), summary.col_spec() now returns early instead of calling min() on an empty vector, which produced a confusing "no non-missing arguments to min" warning.

Fixes #521

Reprex

library(vroom)
foo <- "a,b,c\n1,2,3\n"

# Before: produces warning
# Warning: no non-missing arguments to min; returning Inf
vroom(I(foo), delim = ",", col_select = any_of("d"))

# After: no warning, returns empty tibble
vroom(I(foo), delim = ",", col_select = any_of("d"))

Changes

• R/col_types.R: Add early return in summary.col_spec() when all columns are skipped
• tests/testthat/test-vroom.R: Add test for any_of() with non-existent columns

Tests run

• devtools::test(filter = "vroom"): 514 passed, 0 failed
• devtools::test(filter = "col_types"): 14 passed, 0 failed

Related

• cryptic error messages when col_select has no valid columns #521: cryptic error messages when col_select has no valid columns

[github-actions]

[air] _{reported by reviewdog 🐶}

Suggested change

	result <- vroom(I(foo), delim = ",", col_select = any_of("d"), show_col_types = FALSE)
	result <- vroom(
	I(foo),
	delim = ",",
	col_select = any_of("d"),
	show_col_types = FALSE
	)

[github-actions]

[air] _{reported by reviewdog 🐶}

Suggested change

	result <- vroom(I(foo), delim = ",", col_select = any_of("a"), show_col_types = FALSE)
	result <- vroom(
	I(foo),
	delim = ",",
	col_select = any_of("a"),
	show_col_types = FALSE
	)

[LeonidasZhak]

I am withdrawing this PR as part of a broader cleanup of an oversized automated contribution batch. I am sorry for the review noise and the pressure this may have put on maintainers. Going forward I will keep contributions to this project much more limited and higher-signal, ideally only one or two focused PRs at a time. Thank you for maintaining the project.