Fix lint-changed-files CI for {lintr} v3.2.0

[joshwlambert]

This PR fixes lintr warnings - causing the lint-changed-files workflow to fail - since the release of {lintr} v3.2.0. Most issues detected have been resolved, some have # nolint flags, where deemed better to keep the existing code, these can potentially be resolved and the # nolint flag removed in the future.

Changes include:

• Adding a dot prefix to variables assigned in functions that overwrite functions exported by base R, e.g. .args
• Remove all() from stopifnot() calls as this is done automatically
• Unnest else after return()
• All lines are less than 80 characters
• Use sample.int() instead of sample()

Add # nolint flags for:

• one_call_pipe_linter
• consecutive_mutate_linter

[joshwlambert]

The pkgdown workflow is now failing on this PR. It was passing on main and I've just re-run the workflow on the HEAD of main and it passes, so it is possible it is due to an issue introduced in this PR.

[joshwlambert]

pkgdown::build_site() runs successfully locally, so I'm going to assume/hope this is an intermittent issue with GitHub actions and if not fix in a separate PR.

One step forward, one step backwards...