Differential D2125

CRAN lister: decode R lister output before JSON loading
AbandonedPublic
Actions

Authored by zack on Oct 11 2019, 8:55 AM.

Tags

None

Subscribers

Details

Reviewers

Group Reviewers

Diff Detail

Repository

Branch

bug/cran-stdout-encoding

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 8316
Build 12031: tox-on-jenkins	Jenkins
Build 12030: arc lint + arc unit

Event Timeline

zack created this revision.Oct 11 2019, 8:55 AM

Herald added a reviewer: Reviewers. · View Herald TranscriptOct 11 2019, 8:55 AM

zack added a child revision: D2123: typing: minimal changes to make a no-op mypy run pass.Oct 11 2019, 8:56 AM

Build is green
See https://jenkins.softwareheritage.org/job/DLS/job/tox/419/ for more details.

Harbormaster completed remote builds in B8316: Diff 7114.Oct 11 2019, 8:59 AM

I'll amend D2120 with this if you don't mind.

ardumont accepted this revision.Oct 11 2019, 10:06 AM

This revision is now accepted and ready to land.Oct 11 2019, 10:06 AM

I'll amend D2120 with this if you don't mind.

Or you land it first and i rebase and update that diff, as you wish ;)

ardumont mentioned this in D2120: cran.lister: Fix cran lister and add proper integration test.Oct 11 2019, 10:12 AM

This is now obsolete as that was merged within the D2120 diff which adds integration test and fixes the cran lister behavior.

In D2125#49275, @ardumont wrote:

This is now obsolete as that was merged within the D2120 diff which add integration test and fixes the cran lister behavior.

now that D2120 implements this, I'm dropping this diff

zack removed a child revision: D2123: typing: minimal changes to make a no-op mypy run pass.Oct 11 2019, 10:25 AM

zack mentioned this in D2123: typing: minimal changes to make a no-op mypy run pass.

Revision Contents
Changeset List

Path

Size

swh/

lister/

cran/

2 lines

Diff 7114

swh/lister/cran/lister.py

Loading...