Skip to content

Conversation

@clerousset
Copy link
Contributor

Removed duplicate mention of the '<-' operator and its implications.

@codecov
Copy link

codecov bot commented Oct 28, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.24%. Comparing base (a6ac90b) to head (3a555fb).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2961   +/-   ##
=======================================
  Coverage   99.24%   99.24%           
=======================================
  Files         129      129           
  Lines        7282     7282           
=======================================
  Hits         7227     7227           
  Misses         55       55           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Bisaloo
Copy link
Collaborator

Bisaloo commented Oct 28, 2025

Hi @clerousset, thanks for the contribution!

Files with the .Rd extension under man/ are auto-generated. Generally, if you spot an error here, it should be fixed in the source file. You can find the related source file by looking at the first line of the .Rd file. In this case:

% Generated by roxygen2: do not edit by hand
% Please edit documentation in R/zzz.R

Admittedly, in the case of lintr, there are extra bells and whistles involved in the generation of these .Rd files and pinpointing the exact piece of code or documentation responsible for the problem is not always straightforward.

In this specific case, what we are seeing is a result of a typo in:

lintr/R/zzz.R

Line 233 in 23ebb17

`->>` = "\\link[base:assignOps]{<<-}"

As seen by the element name, we want to refer to ->> but are actually mentioning <<-. The fix would be:

  op_link_map <- c(
    `:::` = "\\link[base:ns-dblcolon]{:::}",
    `<<-` = "\\link[base:assignOps]{<<-}",
-    `->>` = "\\link[base:assignOps]{<<-}"
+   `->>` = "\\link[base:assinOps]{->>}"
  )

Would you like to make the change in this pull request? To then re-generate the updated documentation file (.Rd), you would have to run devtools::document().

Removed duplicate mention of the '<-' operator and its implications.

Fix duplicate line
@clerousset
Copy link
Contributor Author

my bad, thanks. It should be good now

@Bisaloo
Copy link
Collaborator

Bisaloo commented Oct 29, 2025

You need to re-run devtools::document(), which is why the repo-meta-tests check is failing. Could you do this please? This is then ready to be merged.

@Bisaloo Bisaloo merged commit ec332f1 into r-lib:main Nov 1, 2025
21 checks passed
@Bisaloo
Copy link
Collaborator

Bisaloo commented Nov 1, 2025

I went ahead and run devtools::document() on your branch. Thanks again for catching this and submitting a PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants