Differential D8163

api/server: Clarify load and check configuration backend
ClosedPublic
Actions

Authored by ardumont on Jul 29 2022, 10:13 AM.

Details

Reviewers

None

Group Reviewers

Reviewers

Commits

rDSCHfd6df6ab4fd7: api/server: Clarify load and check configuration backend

Summary

This adds type to the function, update its docstring and clarify its associated tests as
well.

not for review

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

ardumont created this revision.Jul 29 2022, 10:13 AM

Herald added a reviewer: Reviewers. · View Herald TranscriptJul 29 2022, 10:13 AM

Build is green

Patch application report for D8163 (id=29480)

Rebasing onto d8474480ae...

Current branch diff-target is up to date.

Changes applied before test

commit fd6df6ab4fd79e2cfa6b33ed39cd68af367ab91e
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Fri Jul 29 10:12:23 2022 +0200

    api/server: Clarify load and check configuration backend
    
    This adds type to the function, update its docstring and clarify its associated tests as
    well.

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

Harbormaster completed remote builds in B30554: Diff 29480.Jul 29 2022, 10:18 AM

ardumont requested review of this revision.Jul 29 2022, 10:18 AM

This revision was not accepted when it landed; it landed in state Needs Review.Jul 29 2022, 10:24 AM

Closed by commit rDSCHfd6df6ab4fd7: api/server: Clarify load and check configuration backend (authored by ardumont). · Explain Why

This revision was automatically updated to reflect the committed changes.

ardumont added a commit: rDSCHfd6df6ab4fd7: api/server: Clarify load and check configuration backend.