-
Notifications
You must be signed in to change notification settings - Fork 188
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 spam-filter module and associated code #415
Comments
Seeing how spam-filter hasn't been updated in 2 years and depends on node 8 and npm 5... I started digging around our packages to see which ones are also in the same boat where it hasn't been updated in a while.
These aren't complaining now but I'm worried that they might complain in the future @_@ |
@cindyledev Thank you for the good point. As she pointed out, this hasn't been updated in 2 years and its dependencies will conflict with the rest of our other libraries. We have decided to remove the spam-filter module and remove any tests associated with the module. |
@humphd are you ok with me assigning it to you so your students can take this issue? |
That, or assign it to yourself, and you can mentor. Either works. |
I'm going to steal this... |
We're having issues using the https://www.npmjs.com/package/spam-filter module, since it depends on a really old version of node. Let's remove it, or replace it.
We're not currently using spam filtering at all, so it's critical to have this (yet).
The text was updated successfully, but these errors were encountered: