launchpad-reviewers team mailing list archive

[Alvaro Crespo Serrano <mp+486923@xxxxxxxxxxxxxxxxxx>]

addressed


Diff comments:

> diff --git a/lib/lp/charms/model/charmrecipejob.py b/lib/lp/charms/model/charmrecipejob.py
> index 56efa07..3b91787 100644
> --- a/lib/lp/charms/model/charmrecipejob.py
> +++ b/lib/lp/charms/model/charmrecipejob.py
> @@ -173,6 +173,14 @@ class CharmRecipeRequestBuildsJob(CharmRecipeJobDerived):
>      @classmethod
>      def create(cls, recipe, requester, channels=None, architectures=None):
>          """See `ICharmRecipeRequestBuildsJobSource`."""
> +        # architectures can be an iterable of strings or Processors
> +        # in the latter case, we need to convert them to strings

applied to all instances

> +        if architectures and all(
> +            not isinstance(arch, str) for arch in architectures
> +        ):
> +            architectures = [
> +                architecture.name for architecture in architectures
> +            ]
>          metadata = {
>              "requester": requester.id,
>              "channels": channels,


-- 
https://code.launchpad.net/~alvarocs/launchpad/+git/launchpad/+merge/486923
Your team Launchpad code reviewers is requested to review the proposed merge of ~alvarocs/launchpad:request-archs-charms into launchpad:master.