-
Notifications
You must be signed in to change notification settings - Fork 33
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 Twitter/tweet analysis #322
Comments
I think removing it makes sense as well: we don't really have the energy and interest to keep that part of Archives Unleashed maintained, and I think keeping AUT on the core mission of "web archive analysis" is a good idea. |
I'd like to keep it, unlike fatjar size is really an issue? Who knows, someone might want to take it up later...? And we have the ambition of doing analysis on webpages mentioned in tweets (and some point in time)? |
@lintool it's in the git history, and we can add it back in if there is a real demand for it. I'd like to get to the bottom on dependency issues, and removing this will help a lot. |
@lintool @ianmilligan1 is there any major reason to keep the Tweet support around? It's not really fully fleshed out, and there are a bunch of dependencies we could clean out if we remove it.
The text was updated successfully, but these errors were encountered: