Webtorrent: Proposal: Deprecate `parse-torrent-file`

Created on 14 Apr 2017  路  4Comments  路  Source: webtorrent/webtorrent

I'm really happy that WebTorrent is hyper-modular, since it's enabled tons of innovation on top of the core modules. There are so many use cases for these lower-level modules that we never could have anticipated!

That said, sometimes we've gone too far.

In the past, webtorrent-tracker and bittorrent-tracker were two separate packages. Now we just have bittorrent-tracker, which was the right move.

We also used to have webtorrent, bittorrent-client, and bittorrent-swarm. Now we've merged them all into webtorrent since the others weren't usable on their own.

I propose that we deprecate parse-torrent-file and merge it into parse-torrent. The test suites for both are almost entirely identical and it's always better to use parse-torrent instead of parse-torrent-file, since it also supports parsing magnet links (just one dep more).

Thoughts?

Most helpful comment

Just merged parse-torrent-file into parse-torrent. Released it as [email protected]. I also marked parse-torrent-file as deprecated.

All 4 comments

cc @webtorrent/members

Voted with a reaction. I invite everyone to do the same.

I believe it's unanimous. @feross are you the one merging them?

Just merged parse-torrent-file into parse-torrent. Released it as [email protected]. I also marked parse-torrent-file as deprecated.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

Lev09 picture Lev09  路  3Comments

bubundas17 picture bubundas17  路  8Comments

Clodo76 picture Clodo76  路  7Comments

kocoten1992 picture kocoten1992  路  6Comments

Weedshaker picture Weedshaker  路  7Comments