Skip to content

Migrate package pipelines from magrittr %>% to native R |> - #24

Open
smwoodman with Copilot wants to merge 4 commits into
devfrom
copilot/update-to-native-pipe-operator
Open

Migrate package pipelines from magrittr %>% to native R |>#24
smwoodman with Copilot wants to merge 4 commits into
devfrom
copilot/update-to-native-pipe-operator

Conversation

Copilot AI commented Jul 31, 2026

Copy link
Copy Markdown

This updates the package to use R’s native pipe operator across code and supporting materials, removing reliance on magrittr pipe syntax. The change is applied package-wide so behavior and style stay consistent.

  • Pipeline syntax migration

    • Replaced %>% with |> in package source under R/
    • Applied the same conversion in tests/, vignettes/, and data-raw/ to keep examples and helpers aligned
  • Package metadata / namespace cleanup

    • Removed magrittr from DESCRIPTION imports
    • Removed magrittr pipe import entries from roxygen package metadata and NAMESPACE
  • Illustrative change

    # before
    y.proc %>%
      filter(Event %in% c("R", "E")) %>%
      select(Event, Lat, Lon)
    
    # after
    y.proc |>
      filter(Event %in% c("R", "E")) |>
      select(Event, Lat, Lon)

Copilot AI changed the title Migrate swfscDAS to native R pipe operator Migrate package pipelines from magrittr %>% to native R |> Jul 31, 2026
Copilot AI requested a review from smwoodman July 31, 2026 03:27
@smwoodman
smwoodman marked this pull request as ready for review July 31, 2026 03:40
@smwoodman
smwoodman force-pushed the copilot/update-to-native-pipe-operator branch from bf24f9c to 3fb8986 Compare July 31, 2026 03:43
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