Skip to content
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

Closed
ruebot opened this issue Jul 8, 2019 · 3 comments

Comments

Projects
None yet
3 participants
@ruebot
Copy link
Member

commented Jul 8, 2019

@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.

ruebot added a commit that referenced this issue Jul 8, 2019

Remove Tweet utils.
- Resolves #322
- Resolves #206
- Resolves #194
@ianmilligan1

This comment has been minimized.

Copy link
Member

commented Jul 8, 2019

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.

@lintool

This comment has been minimized.

Copy link
Member

commented Jul 8, 2019

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)?

@ruebot

This comment has been minimized.

Copy link
Member Author

commented Jul 8, 2019

@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.

ianmilligan1 added a commit that referenced this issue Jul 15, 2019

Remove Tweet utils. (#323)
- Resolves #322
- Resolves #206
- Resolves #194
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.