Skip to content

Remove magrittr dependency and replace %>% with native pipe |>#341

Merged
venpopov merged 2 commits intodevelopfrom
copilot/remove-magrittr-and-replace-pipe
Mar 4, 2026
Merged

Remove magrittr dependency and replace %>% with native pipe |>#341
venpopov merged 2 commits intodevelopfrom
copilot/remove-magrittr-and-replace-pipe

Conversation

Copy link
Contributor

Copilot AI commented Mar 2, 2026

Summary

Since R ≥ 4.1.0 is already the minimum version, replace all magrittr %>% with native |> and drop magrittr from Suggests.

  • DESCRIPTION: Remove magrittr from Suggests
  • R source (distributions.R, utils.R): Update pipes in roxygen example and user-facing message
  • Vignettes (5 files): %>%|> throughout
  • Dev/test scripts (.dev/, tests/internal/): Same replacement

Notable: in bmm_m3.Rmd, magrittr's . pronoun (.$data) replaced with explicit m3_fit$data since native pipe doesn't support dot forwarding.

Tests

[] Confirm that all tests passed
[] Confirm that devtools::check() produces no errors

Release notes

Remove magrittr as a suggested dependency. All pipe usage now uses R's native |> operator.

Original prompt

This section details on the original issue you should resolve

<issue_title>Remove magritte suggest and replace with native pipe everywhere</issue_title>
<issue_description>since we bump to R 4.1 in #339 we can now use the native pipe. Magrittr is a suggested dependency because we only use the pipes in vignettes, but we should remove it completely and replace with native pipe</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: venpopov <12783882+venpopov@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove magrittr and replace with native pipe Remove magrittr dependency and replace %>% with native pipe |> Mar 2, 2026
@venpopov venpopov marked this pull request as ready for review March 3, 2026 00:02
@venpopov venpopov merged commit 3cadc38 into develop Mar 4, 2026
3 checks passed
@venpopov venpopov deleted the copilot/remove-magrittr-and-replace-pipe branch March 4, 2026 00:37
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.

Remove magritte suggest and replace with native pipe everywhere

2 participants