-
Notifications
You must be signed in to change notification settings - Fork 132
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove tidyr::spread()
#552
Conversation
Codecov Report
@@ Coverage Diff @@
## main #552 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 26 26
Lines 1179 1183 +4
=========================================
+ Hits 1179 1183 +4
|
Excellent! Thank you for commenting the most complex section. And nice job catching the instance of "spread" that is in a comment, not code 😁 |
BTW did the auto-linter run catch the whitespaces that prompted your last commit? |
Hi, sfirke, it did, in the sense that I get a notification that the style action didn't pass. See #549 (comment) |
Description
Removing mentions of the superseded tidyr functions.
Related Issue
Addresses #549