Skip to content

Conversation

@mattfidler
Copy link
Member

Seems that posit is depreciating magrittr with lintr

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR migrates from magrittr's pipe operator (%>%) to R's native pipe operator (|>) throughout the codebase, aligning with Posit's deprecation of magrittr in lintr. The changes are mechanical replacements with no functional modifications to the code logic.

Key Changes:

  • Replaced all instances of %>% with |> across vignettes and test files
  • Removed trailing whitespace in several files
  • Maintained all existing functionality and code structure

Reviewed changes

Copilot reviewed 130 out of 130 changed files in this pull request and generated no comments.

Show a summary per file
File Description
vignettes/rxode2-plot.Rmd Updated pipe operators in event table examples and removed trailing spaces
vignettes/rxode2-event-types.Rmdh Updated pipe operators throughout dosing and infusion examples, cleaned up whitespace
vignettes/events-et.Rmdh Updated pipe operators in event table construction examples
vignettes/df.Rmdh Updated pipe operators in data manipulation examples
vignettes/articles/Modifying-Models.Rmd Updated pipe operators in model piping examples
tests/testthat/test-zzzz-plot.R Updated pipe operators in all plotting test cases
tests/testthat/test-ui-*.R Updated pipe operators across multiple UI-related test files
tests/testthat/test-*.R Updated pipe operators across all remaining test files

The changes are consistent and complete. All instances of the magrittr pipe have been properly replaced with R's native pipe operator. No functional issues or inconsistencies were identified.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@mattfidler mattfidler merged commit c8d6640 into main Dec 29, 2025
6 of 8 checks passed
@mattfidler mattfidler deleted the new-pipe branch December 29, 2025 02:22
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