search

carpentries-incubator / targets-workshop

Pre-alpha {targets} workshop
https://carpentries-incubator.github.io/targets-workshop/
Other
33 stars 6 forks source link

switch to base pipe #4

Closed drmowinckels closed 1 year ago

drmowinckels commented 1 year ago

Its generally recommended to switch teaching pipes to the base pipe rather than the magrittr pipe, as only quite special and advanced use in magrittr pipe makes a difference.

github-actions[bot] commented 1 year ago

:ok: Pre-flight checks passed :smiley:

This pull request has been checked and contains no modified workflow files or spoofing.

It should be safe to Approve and Run the workflows that need maintainer approval.

joelnitta commented 1 year ago

Is there actually a change in the code to base pipe? I don't see any changes in the files.

joelnitta commented 1 year ago

Fixed by https://github.com/joelnitta/targets-workshop/pull/5