Differential D8384

npm, maven: ignore blatantly invalid licenses
ClosedPublic
Actions

Authored by vlorentz on Sep 2 2022, 1:53 PM.

Details

Reviewers

anlambert

Group Reviewers

Reviewers

Commits

rDCIDX44879ab56317: npm, maven: ignore blatantly invalid licenses

Summary

They cause noisy logs

Diff Detail

Repository

rDCIDX Metadata indexer

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 31295
Build 48954: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 48953: arc lint + arc unit

Event Timeline

vlorentz created this revision.Sep 2 2022, 1:53 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptSep 2 2022, 1:53 PM

Build is green

Patch application report for D8384 (id=30257)

Rebasing onto b056d431fd...

Current branch diff-target is up to date.

Changes applied before test

commit 44879ab5631784a4b5c02e66d8818e797390d9a6
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Sep 2 13:52:53 2022 +0200

    npm, maven: ignore blatantly invalid licenses
    
    They cause noisy logs

See https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/479/ for more details.

Harbormaster completed remote builds in B31295: Diff 30257.Sep 2 2022, 2:01 PM

vlorentz requested review of this revision.Sep 2 2022, 2:01 PM

anlambert accepted this revision.Sep 2 2022, 2:39 PM

This revision is now accepted and ready to land.Sep 2 2022, 2:39 PM

Closed by commit rDCIDX44879ab56317: npm, maven: ignore blatantly invalid licenses (authored by vlorentz). · Explain WhySep 2 2022, 2:50 PM

This revision was automatically updated to reflect the committed changes.

vlorentz added a commit: rDCIDX44879ab56317: npm, maven: ignore blatantly invalid licenses.

Revision Contents
Changeset List

Path

Size

Coverage (All)

Coverage (Touched)

swh/

indexer/

metadata_dictionary/

maven.py

3 lines

94%

npm.py

6 lines

95%

tests/

metadata_dictionary/

test_maven.py

41 lines

100%

test_npm.py

18 lines

100%

Diff 30257

View Options

swh/indexer/metadata_dictionary/maven.py

View Options

swh/indexer/metadata_dictionary/npm.py

View Options

swh/indexer/tests/metadata_dictionary/test_maven.py

View Options

swh/indexer/tests/metadata_dictionary/test_npm.py

npm, maven: ignore blatantly invalid licensesClosedPublicActions

Details

Diff Detail

Event Timeline

Patch application report for D8384 (id=30257)

Changes applied before test

Revision ContentsChangeset List

Diff 30257

swh/indexer/metadata_dictionary/maven.py

swh/indexer/metadata_dictionary/npm.py

swh/indexer/tests/metadata_dictionary/test_maven.py

swh/indexer/tests/metadata_dictionary/test_npm.py

npm, maven: ignore blatantly invalid licenses
ClosedPublic
Actions

Revision Contents
Changeset List