Skip to content

Cleanup deprecated libraries#112

Open
tancnle wants to merge 2 commits into
johnmichel:mainfrom
tancnle:cleanup-deprecated-libraries
Open

Cleanup deprecated libraries#112
tancnle wants to merge 2 commits into
johnmichel:mainfrom
tancnle:cleanup-deprecated-libraries

Conversation

@tancnle

@tancnle tancnle commented Jun 1, 2018

Copy link
Copy Markdown
Contributor

Description

Remove no-longer maintained/deprecated libraries

/cc @johnmichel

@tancnle

tancnle commented Dec 26, 2018

Copy link
Copy Markdown
Contributor Author

@johnmichel Is there any chance you can have a look at this PR? It has been a while.

@Semigradsky

Copy link
Copy Markdown

Why you want to remove them?

@wumpus

wumpus commented Feb 1, 2019

Copy link
Copy Markdown

Removing obsolete libs from the detector makes web archives people sad.

@tancnle

tancnle commented Apr 19, 2019

Copy link
Copy Markdown
Contributor Author

The motivations are:

  • Some of those URL references no longer exist (i.e. clicking on library icon results on 404 page)
  • Reduce libraries.js size

@nettrino

nettrino commented Nov 1, 2021

Copy link
Copy Markdown

+1 on not removing deprecated libraries. This information is useful as it lets us know of websites without outdated dependencies and/or security issues. It would actually be useful to have a special icon next to the library to denote its deprecated. For the purposes of this PR perhaps we could turn this into an option where the user can choose to load or not load deprecated libraries?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants