search

rocker-org / devcontainer-features

A collection of Dev Container Features.
https://rocker-project.org/images/devcontainer/features.html
MIT License
52 stars 16 forks source link

[r-packages] allow whitespace in `packages` #207

Closed dpprdan closed 1 year ago

dpprdan commented 1 year ago

This is a small change to allow for whitespace in the packages specification, so that using

"packages": "cli, rlang"

does not lead to an error.

dpprdan commented 1 year ago

@eitsupi Is the test correct like this? I am not sure.

I'll add a similar test for the apt-packages feature. It would only be consistent if it also allowed for whitespace in the package list. It already does IIUC, but a test might be useful to prevent a regression? Feel free to revert, if this doesn't make sense.