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
#547 implements a missing_package_linter() which checks if all packages loaded by library(), require(), loadNamespace(), and requireNamespace() are installed.
Apart from that, we could also check the usage of ns::obj and ns:::obj as they also used to access objects in a namespace (provided by package).
We could implement a separate linter (as suggested at #547 (comment)) to check if
The package ns in ns::obj and ns:::obj are installed.
The obj in ns::obj is exported from namespace ns.
The obj in ns:::obj exists in namespace ns.
While checking 1 does not load any new package, checking 2 and 3 might load the package being checked. I'm not sure if we allow loading extra packages from user code?
The text was updated successfully, but these errors were encountered:
#547 implements a
missing_package_linter()
which checks if all packages loaded bylibrary()
,require()
,loadNamespace()
, andrequireNamespace()
are installed.Apart from that, we could also check the usage of
ns::obj
andns:::obj
as they also used to access objects in a namespace (provided by package).We could implement a separate linter (as suggested at #547 (comment)) to check if
ns
inns::obj
andns:::obj
are installed.obj
inns::obj
is exported from namespacens
.obj
inns:::obj
exists in namespacens
.While checking 1 does not load any new package, checking 2 and 3 might load the package being checked. I'm not sure if we allow loading extra packages from user code?
The text was updated successfully, but these errors were encountered: