search

safer-r / saferDev

GNU General Public License v3.0
0 stars 0 forks source link

Problem of ERROR message display for arguments with no default value #13

Closed gael-millot closed 6 months ago

gael-millot commented 6 months ago

The problem is one missing space in the message saferDev::get_message() -> FOLLOWING ARGUMENTHAS NO DEFAULT VALUE AND REQUIRE ONE:

So check and replace in every function of all packages: base::ifelse(base::sum(tempo, na.rm = TRUE) > 1, "S HAVE", "HAS"), " NO DEFAULT By base::ifelse(base::sum(tempo, na.rm = TRUE) > 1, "S HAVE", " HAS"), " NO DEFAULT

Tintin2710 commented 6 months ago

saferDev done

gael-millot commented 6 months ago

Thanks ! Il faudrait le faire pour tous les packages. Ok ?

Tintin2710 commented 6 months ago

OK！

发送自 Outlook for iOShttps://aka.ms/o0ukef

发件人: gael-millot @.> 发送时间: Wednesday, May 15, 2024 4:04:25 PM 收件人: safer-r/saferDev @.> 抄送: Haiding WANG @.>; State change @.> 主题: Re: [safer-r/saferDev] Problem of ERROR message display for arguments with no default value (Issue #13)

Thanks ! Il faudrait le faire pour tous les packages. Ok ?

― Reply to this email directly, view it on GitHubhttps://github.com/safer-r/saferDev/issues/13#issuecomment-2112641669, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BFXRVXYQE6R7FNAPCM74XK3ZCNTOTAVCNFSM6AAAAABHYF2I5WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJSGY2DCNRWHE. You are receiving this because you modified the open/close state.Message ID: @.***>