Join GitHub today
GitHub is home to over 36 million developers working together to host and review code, manage projects, and build software together.
Sign upRemove Twitter/tweet analysis #322
Comments
ruebot
added a commit
that referenced
this issue
Jul 8, 2019
ruebot
referenced a pull request that will
close
this issue
Jul 8, 2019
Open
Remove Tweet utils. #323
This comment has been minimized.
This comment has been minimized.
|
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. |
This comment has been minimized.
This comment has been minimized.
|
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)? |
This comment has been minimized.
This comment has been minimized.
|
@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. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
ruebot commentedJul 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.