Differential D5513

Fix argument syntax to remove warnings and use Napoleon
ClosedPublic
Actions

Authored by vlorentz on Apr 13 2021, 9:55 PM.

Details

Reviewers

ardumont

Group Reviewers

Reviewers

Commits

rDLDHGf03f274065cd: Fix argument syntax to remove warnings and use Napoleon

Diff Detail

Repository

rDLDHG Mercurial loader

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 20702
Build 32128: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 32127: arc lint + arc unit

Event Timeline

vlorentz created this revision.Apr 13 2021, 9:55 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptApr 13 2021, 9:55 PM

Build is green

Patch application report for D5513 (id=19674)

Rebasing onto cc04dd0bd0...

Current branch diff-target is up to date.

Changes applied before test

commit f03f274065cd4a643786205fc2a535e9270a0f9c
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Apr 13 21:55:33 2021 +0200

    Fix argument syntax to remove warnings and use Napoleon

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

Harbormaster completed remote builds in B20702: Diff 19674.Apr 13 2021, 9:57 PM

vlorentz requested review of this revision.Apr 13 2021, 9:57 PM

ardumont accepted this revision.Apr 14 2021, 10:36 AM

This revision is now accepted and ready to land.Apr 14 2021, 10:36 AM

Closed by commit rDLDHGf03f274065cd: Fix argument syntax to remove warnings and use Napoleon (authored by vlorentz). · Explain WhyApr 15 2021, 10:20 AM

This revision was automatically updated to reflect the committed changes.

vlorentz added a commit: rDLDHGf03f274065cd: Fix argument syntax to remove warnings and use Napoleon.

Revision Contents
Changeset List

Path

Size

Coverage (All)

Coverage (Touched)

swh/

loader/

mercurial/

identify.py

29 lines

93%

Diff 19674

View Options

swh/loader/mercurial/identify.py

Fix argument syntax to remove warnings and use NapoleonClosedPublicActions