[
https://issues.apache.org/jira/browse/ARROW-15359?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Dragoș Moldovan-Grünfeld updated ARROW-15359:
---------------------------------------------
Description:
Currently we have a mixture of {{base::stop()}} and {{rlang::abort()}} in the
codebase. Should we make a deliberate choice to use one or the other?
If we decided to switch to {{rlang}}, we could:
* change the codebase as we go along, switching from {{base}} functions
({{stop(), warning(), message()}} to their {{rlang}} equivalents {{abort(),
warn(), inform()}}
* make an addendum to {{STYLE.md}} on condition handling
Note: the {{arrow}} R package already depends on {{rlang}} so any outcome will
not result in additional dependencies.
was:
Currently we have a mixture of {{base::stop()}} and {{rlang::abort()}} in the
codebase. Should we make a deliberate choice to use one or the other?
If we decided to switch to {{rlang}}, we could:
* change the codebase as we go along, switching from {{base}} functions
({{stop(), warning(), message()}} to their {{rlang}} equivalents {{abort(),
warn(), inform()}}
* make an addendum to {{STYLE.md}} on condition handling
> [R] Discussion: should we use {rlang} for condition handling
> ------------------------------------------------------------
>
> Key: ARROW-15359
> URL: https://issues.apache.org/jira/browse/ARROW-15359
> Project: Apache Arrow
> Issue Type: Improvement
> Components: R
> Reporter: Dragoș Moldovan-Grünfeld
> Priority: Major
>
> Currently we have a mixture of {{base::stop()}} and {{rlang::abort()}} in the
> codebase. Should we make a deliberate choice to use one or the other?
> If we decided to switch to {{rlang}}, we could:
> * change the codebase as we go along, switching from {{base}} functions
> ({{stop(), warning(), message()}} to their {{rlang}} equivalents {{abort(),
> warn(), inform()}}
> * make an addendum to {{STYLE.md}} on condition handling
> Note: the {{arrow}} R package already depends on {{rlang}} so any outcome
> will not result in additional dependencies.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
