search

OpenDendro / dplR

This is the dev site for the dplR package in R
38 stars 14 forks source link

Resolving R CMD check NOTE #23

Closed mvkorpel closed 3 weeks ago

mvkorpel commented 4 months ago

Recent R-devel complained about dplR using a non-API entry point. This patch resolves the NOTE. The solution, including a backport for R older than 4.4.1, was adopted from the Writing R Extensions document.

In an unrelated change, a check for R older than 3.0.0 was removed.

mvkorpel commented 4 months ago

I also tested this with R 4.3.0 where the backported allocLang() kicks in.

AndyBunn commented 4 months ago

Thanks so much Mikko! I’ll look at this tomorrow.

From: Mikko Korpela @.> Date: Friday, July 12, 2024 at 1:39 PM To: OpenDendro/dplR @.> Cc: Andy Bunn @.>, Review requested @.> Subject: Re: [OpenDendro/dplR] Resolving R CMD check NOTE (PR #23)

I also tested this with R 4.3.0 where the backported allocLang() kicks in.

— Reply to this email directly, view it on GitHubhttps://github.com/OpenDendro/dplR/pull/23#issuecomment-2226321694, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AC7UCXJOILTDWOFYGLXMFL3ZMA5IFAVCNFSM6AAAAABKZQQFV6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRWGMZDCNRZGQ. You are receiving this because your review was requested.Message ID: @.***>