Page MenuHomeSoftware Heritage

Add support for content-negotiation in the API
ClosedPublic

Authored by douardda on Jan 29 2019, 3:01 PM.

Diff Detail

Repository
rDSCH Scheduling utilities
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

vlorentz added a subscriber: vlorentz.
vlorentz added inline comments.
swh/scheduler/api/server.py
45–48

Can be rewritten as:

def has_no_empty_params(rule):
    return len(rule.defaults or ()) >= len(rule.arguments or ())
171–183

What is this new endpoint for?

This revision is now accepted and ready to land.Jan 29 2019, 4:52 PM
vlorentz added inline comments.
swh/scheduler/api/server.py
171–183

And it lacks a test.

This revision now requires changes to proceed.Jan 29 2019, 4:53 PM
douardda marked 3 inline comments as done.

Extract the /site-map endpoint in a dedicated diff

sorry part of this diff has been mistakenly push as 94258bda1846

This revision is now accepted and ready to land.Jan 30 2019, 11:48 AM
This revision was automatically updated to reflect the committed changes.