[bug#74124] [PATCH] gnu: Add r-shinyvalidate.

["King, Spencer via Guix-patches" via <guix-patches@gnu.org>]

[-- Attachment #1: Type: text/plain, Size: 1292 bytes --]

Hi Ricardo,

Thank you for the quick response. Is r-testthat now necessary across the board for all R packages? I do have a couple of other outstanding R patches that I can update if needed and ping the team.

Best,
Spencer
________________________________
From: Ricardo Wurmus <rekado@elephly.net>
Sent: Friday, January 3, 2025 12:20 PM
To: 74124-done@debbugs.gnu.org <74124-done@debbugs.gnu.org>
Cc: King, Spencer <spencer.king@wustl.edu>; laurent.gatto@gmail.com <laurent.gatto@gmail.com>; zimon.toutoune@gmail.com <zimon.toutoune@gmail.com>
Subject: [PATCH] gnu: Add r-shinyvalidate.

Hi Spencer,

> Pinging the R team since this patch has been unreviewed for a couple
> months. This is my first time pinging a specific team so please let me
> know if this is inappropriate or if there is something else I should
> do in the future.

My apologies for the delay.  I had not previously seen this patch.
Pinging the team (even much sooner than you did) is certainly the right
thing to do.  It is also recommended to Cc the team when submitting the
patch.  (I don't know if you've done that.)

I've just applied the patch and made a few minor changes, including the
addition of r-testthat, which is now necessary with changes to the
r-build-system.

--
Ricardo

[-- Attachment #2: Type: text/html, Size: 3078 bytes --]