Page MenuHomeSoftware Heritage

exclude test resources from radon tests
ClosedPublic

Authored by olasd on Mar 30 2020, 8:00 PM.

Details

Summary

There's a python file with invalid syntax in there, which confuses it

Test Plan

jenkins stops complaining about radon cc --xml

Diff Detail

Repository
rDWAPPS Web applications
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 11480
Build 17401: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 17400: arc lint + arc unit

Event Timeline

Build is green

Patch application report for D2930 (id=10399)

Rebasing onto d4446bcac7...

Current branch diff-target is up to date.
Changes applied before test
commit 3baabb94d659621d513d4143ad8e2efa8dccc1db
Author: Nicolas Dandrimont <nicolas@dandrimont.eu>
Date:   Mon Mar 30 19:57:26 2020 +0200

    exclude test resources from radon tests
    
    There's a python file with invalid syntax in there, which confuses it

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

Build is green

Patch application report for D2930 (id=10400)

Rebasing onto d4446bcac7...

Current branch diff-target is up to date.
Changes applied before test
commit 286869712d2d261fbe368907173bef69501b760f
Author: Nicolas Dandrimont <nicolas@dandrimont.eu>
Date:   Mon Mar 30 19:57:26 2020 +0200

    exclude test resources from radon tests
    
    There's a python file with invalid syntax in there, which confuses it

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

This revision is now accepted and ready to land.Mar 30 2020, 8:16 PM

Build is green

Patch application report for D2930 (id=10405)

Rebasing onto 5f248129e1...

Current branch diff-target is up to date.
Changes applied before test
commit 9aa7f44e8105453a8ec76669c19c90a97a27f50d
Author: Nicolas Dandrimont <nicolas@dandrimont.eu>
Date:   Mon Mar 30 19:57:26 2020 +0200

    exclude test resources from radon tests
    
    There's a python file with invalid syntax in there, which confuses it

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

This revision was automatically updated to reflect the committed changes.