-
Notifications
You must be signed in to change notification settings - Fork 49
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 cruft #384
Remove cruft #384
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would leave some additional breadcrumbs about the changes and the removed features and packages.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Glad to see old code removed! I think we probably want to rename @snoopi_deep
to @snoopi
, while also deprecating @snoopi_deep
and making it an alias for the new @snoopi
?
How about a more wholesale renaming?
|
That sounds good to me. I'm all for that. |
This eliminates `@snoopc` and `@snoopi`, as these are legacy tools that aren't needed anymore.
fba3a68
to
07bcb2a
Compare
Further changes, like renaming the main data-collection macros, are things I intend for a future PR. |
This certainly isn't complete (e.g., the documentation will be updated separately), but it's a big chunk of #382: