search

exercism / r

Exercism exercises in R.
https://exercism.org/tracks/r
MIT License
27 stars 38 forks source link

Add `run-length-encoding` exercise #365

Closed ErikSchierboom closed 1 month ago

colinleach commented 1 month ago

This looks like a problem with the way CI is set up, probably?

I just checked in the Exercism online editor, and that is happy with library(stringr).

colinleach commented 1 month ago

ci.yml:

        run: |
          install.packages(c("testthat", "jsonlite", "lintr", "dplyr"))

Do we need to add "tidyverse" to match last year's changes to the test runner? This really isn't my expertise!

ErikSchierboom commented 1 month ago

Do we need to add "tidyverse" to match last year's changes to the test runner? This really isn't my expertise!

We do. I'll fix this tomorrow!

colinleach commented 1 month ago

Given our 9h time difference, I went ahead and attempted the fix: PR #368.

@ErikSchierboom, can you review this? You'll see that I had a couple of attempts, deciding along the way that we don't need all of tidyverse for Exercism purposes.