Diff comments:
> diff --git a/lib/lp/bugs/scripts/svthandler.py
> b/lib/lp/bugs/scripts/svthandler.py
> index 9e9d096..9907105 100644
> --- a/lib/lp/bugs/scripts/svthandler.py
> +++ b/lib/lp/bugs/scripts/svthandler.py
> @@ -20,6 +20,10 @@ class SVTRecord:
> """Parse a string and return a SVTRecord."""
> raise NotImplementedError()
>
> + def to_yaml(self) -> str:
Agree, in that case including `yaml` in the method name does not make sense if
it's not yaml.
> + """Convert the SVTRecord to a YAML string."""
> + raise NotImplementedError()
> +
>
> class SVTImporter:
> def from_record(
--
https://code.launchpad.net/~ines-almeida/launchpad/+git/launchpad/+merge/493398
Your team Launchpad code reviewers is requested to review the proposed merge of
~ines-almeida/launchpad:uct-export-handler into launchpad:master.
_______________________________________________
Mailing list: https://launchpad.net/~launchpad-reviewers
Post to : [email protected]
Unsubscribe : https://launchpad.net/~launchpad-reviewers
More help : https://help.launchpad.net/ListHelp
