2 small improvements
ClosedPublic
Actions

Authored by douardda on Jan 31 2019, 4:04 PM.

Details

Reviewers

Group Reviewers

Commits

rDCORE657819435b1c: Fix async version of decode_request
rDCORE508e3d0d15df: Make the cur_arg argument optional in BaseDb.cursor()

Summary

Make the cur_arg argument optional in BaseDb.cursor() just to make our life a bit easier...

Fix async version of decode_request so thath it does not fail if no payload at all is provided in the request, eg. for an endpoint which really is (or at least should be) a simple GET...

Diff Detail

Repository

rDCORE Foundations and core functionalities

Branch

master

Lint

Lint Skipped

Unit

Unit Tests Skipped

Build Status

Buildable 3907
Build 5118: tox-on-jenkins	Jenkins
Build 5117: arc lint + arc unit

Event Timeline

douardda created this revision.Jan 31 2019, 4:04 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptJan 31 2019, 4:04 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DCORE/job/tox/136/
See console output for more information: https://jenkins.softwareheritage.org/job/DCORE/job/tox/136/console

Harbormaster failed remote builds in B3907: Diff 3323!Jan 31 2019, 4:05 PM

ardumont accepted this revision.Jan 31 2019, 4:07 PM

This revision is now accepted and ready to land.Jan 31 2019, 4:07 PM

Closed by commit rDCORE508e3d0d15df: Make the cur_arg argument optional in BaseDb.cursor() (authored by douardda). · Explain WhyJan 31 2019, 5:24 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

swh/

core/

api/

asynchronous.py

3 lines

db/

__init__.py

2 lines

Diff 3323

View Options

swh/core/api/asynchronous.py

View Options

2 small improvementsClosedPublicActions