Closed tibbles-and-tribbles closed 11 months ago
Yes, that makes more sense.
On Dec 13, 2023, at 1:09 PM, Robin Lovelace @.***> wrote:
@Robinlovelace requested changes on this pull request.
In 03-attribute-operations.Rmd https://github.com/geocompx/geocompr/pull/1014#discussion_r1425887110:
@@ -787,7 +787,7 @@ These and other typical raster processing operations are part of the map algebra
Some function names clash between packages (e.g., a function with the name `extract()` exist in both **terra** and **tidyr** packages). This may lead to unexpected results when loading packages in a different order. -In addition to not loading packages by referring to functions verbosely (e.g., `tidyr::extract()`), another way to prevent function names clashes is by unloading the offending package with `detach()`. +In addition to loading packages by referring to functions verbosely (e.g., `tidyr::extract()`), another way to prevent function name clashes is by unloading the offending package with `detach()`. ⬇️ Suggested change -In addition to loading packages by referring to functions verbosely (e.g., `tidyr::extract()`), another way to prevent function name clashes is by unloading the offending package with `detach()`. +In addition to calling functions verbosely with their full namespace (e.g., `tidyr::extract()`) to avoid attaching packages with `library()`, another way to prevent function name clashes is by unloading the offending package with `detach()`. Make more sense?
— Reply to this email directly, view it on GitHub https://github.com/geocompx/geocompr/pull/1014#pullrequestreview-1780493803, or unsubscribe https://github.com/notifications/unsubscribe-auth/A6D75UXA3PJ3AXXV23OGPXLYJIKQBAVCNFSM6AAAAABATKMVQOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTOOBQGQ4TGOBQGM. You are receiving this because you authored the thread.
Removing "not" expresses the desired intent, yes?