/swh/loader/package/npm/loader.py
41	Probably needs a mention of `npm` (in that another loader may interpret an identical source archive differently?)
/swh/loader/package/pypi/loader.py
37	Same as npm, this probably needs to mention `pypi`.

This commit now has outstanding concerns.Mar 23 2021, 6:09 PM

Indeed. I discarded this idea, but can't remember why...

vlorentz mentioned this in D5314: cran, npm, pypi: Add the loader name in the ExtID type.Mar 23 2021, 6:29 PM

Changes (11)

Path

Size

swh/

loader/

package/

archive/

tests/

cran/

debian/

nixguix/

npm/

tests/

pypi/

tests/

tests/

rDLDBASEe9a8f986809c

View Options

swh/loader/package/archive/loader.py

View Options

swh/loader/package/archive/tests/test_archive.py

View Options

swh/loader/package/cran/loader.py

View Options

swh/loader/package/debian/loader.py

View Options

swh/loader/package/loader.py

View Options

swh/loader/package/nixguix/loader.py

View Options

swh/loader/package/npm/loader.py

View Options

swh/loader/package/npm/tests/test_npm.py

View Options

swh/loader/package/pypi/loader.py

View Options

swh/loader/package/pypi/tests/test_pypi.py

View Options

swh/loader/package/tests/test_loader.py

package loaders: define extid typesConcern Raisede9a8f986809cActions

Description

Details

Event Timeline

Changes (11)

rDLDBASEe9a8f986809c

swh/loader/package/archive/loader.py

swh/loader/package/archive/tests/test_archive.py

swh/loader/package/cran/loader.py

swh/loader/package/debian/loader.py

swh/loader/package/loader.py

swh/loader/package/nixguix/loader.py

swh/loader/package/npm/loader.py

swh/loader/package/npm/tests/test_npm.py

swh/loader/package/pypi/loader.py

swh/loader/package/pypi/tests/test_pypi.py

swh/loader/package/tests/test_loader.py

package loaders: define extid types
Concern Raisede9a8f986809c
Actions