You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I thought I already filed this, but I cannot find it. Sorry if it's a reiterated request.
The problem we have with box is that the imported symbols are not explicitly bound. This angers linters, that see a symbol appear out of thin air and start complaining.
Second this, as I have to disable the OBJECT_USAGE_LINTER from {lintr} in codebases using {box} for this reason. However, wondering if support for box::use() syntax could be incorporated into {lintr} to maintain the current syntax, as I think that matches syntax extremely common in Python and many other languages.
I thought I already filed this, but I cannot find it. Sorry if it's a reiterated request.
The problem we have with box is that the imported symbols are not explicitly bound. This angers linters, that see a symbol appear out of thin air and start complaining.
Example
This would trigger a linter problem because hello_world appears from nowhere.
I propose to support the additional syntax
This has the added benefit of allowing for easy renaming of the module in case of collision.
The text was updated successfully, but these errors were encountered: