search

codewars / docs

The Codewars Docs :construction: WIP
https://docs.codewars.com
MIT License
56 stars 201 forks source link

Add R basic setup #406

Closed ggorlen closed 2 years ago

ggorlen commented 2 years ago

@hobovsky Let me know if you'd prefer not to be requested for reviews for this sort of PR, and/or if there's another user that might be interested in hopping in. Happy to have your input though!

hobovsky commented 2 years ago

I will be more than happy to provide any feedback, just let me know when you'd want me to stop :) Please also mind that I am not familiar with this many languages as some other users, and that I live in another timezone so it can take some time until I get to review a freshly published PR. As for other users, you can use language rank leaderboards to get idea who's proficient with what language. Unnamed is always a safe bet, this guy was born and raised in a programming equivalent of the Tower of Babel.

kazk commented 2 years ago

Leaderboard for R: https://www.codewars.com/api/v1/leaders/ranks/r

(I haven't added UI because it's inaccurate due to a known issue preventing contributors from getting rank scores after contributing, but it's usable.)

hobovsky commented 2 years ago

Maybe it would be useful to have a link to the framework docs (https://testthat.r-lib.org/) so users could explore further if necessary? And a version number of the testing framework as well? Or you think it's not necessary, or an additional maintenance problem?

ggorlen commented 2 years ago

Maybe it would be useful to have a link to the framework docs (https://testthat.r-lib.org/) so users could explore further if necessary? And a version number of the testing framework as well? Or you think it's not necessary, or an additional maintenance problem?

I'm flexible--the link is a nice gesture and might save users a search, but if we do decide to include it, we'd want to be consistent and apply to all testing libraries. It's a bit more maintenance, and I'm not sure the benefit is huge.

kazk commented 2 years ago

https://docs.codewars.com/languages/r has a basic setup already (minus context) and a link to their GitHub repo. (I'm not sure if any R kata are using preloaded code, but I want to change the preloaded file to preloaded.R to be more consistent.)

Maybe we should just remove stub pages until we have something.

kazk commented 2 years ago

We can also move that section to this page. I guess that's better.

ggorlen commented 2 years ago

We can also move that section to this page. I guess that's better.

I think so, so it's consistent with languages that have multiple test libraries.