Use inherits() vs class() in R package to pass new CRAN checks

[exalate-issue-sync]

{quote}Flavor: r-devel-linux-x86_64-debian-gcc, r-devel-windows-x86_64
Check: CRAN incoming feasibility, Result: Note_to_CRAN_maintainers
 Maintainer: 'Erin LeDell <[erin@h2o.ai|mailto:erin@h2o.ai]>'

Flavor: r-devel-windows-x86_64
Check: installed package size, Result: NOTE
   installed size is 181.1Mb
   sub-directories of 1Mb or more:
     R       1.4Mb
     java  178.3Mb

Flavor: r-devel-linux-x86_64-debian-gcc, r-devel-windows-x86_64
Check: R code for possible problems, Result: NOTE
 Found if() conditions comparing class() to string:
 File 'h2o/R/communication.R': if (class(rawResponse) == "try-error") ...
 File 'h2o/R/explain.R': if (class(object) == "list") ...
 File 'h2o/R/kvstore.R': if (class(model) == "H2OAutoML") ...
 File 'h2o/R/kvstore.R': if (class(model) == "H2OAutoML") ...
 File 'h2o/R/models.R': if (class(node) == "H2OLeafNode") ...
 Use inherits() (or maybe is()) instead.{quote}

[exalate-issue-sync]

Michal Kurka commented: 1 PR merged, another to follow - [https://github.com//pull/6136#issuecomment-1092299759|https://github.com//pull/6136#issuecomment-1092299759]

[h2o-ops]

JIRA Issue Details

Jira Issue: PUBDEV-8649
Assignee: Erin LeDell
Reporter: Erin LeDell
State: Resolved
Fix Version: 3.36.1.1
Attachments: N/A
Development PRs: Available

[h2o-ops]

Linked PRs from JIRA

#6136