launchpad-reviewers team mailing list archive

[Simone Pelosi <mp+446773@xxxxxxxxxxxxxxxxxx>]

Simone Pelosi has proposed merging ~pelpsi/launchpad:remove-f-string into launchpad:master.

Commit message:
Removed f-string

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~pelpsi/launchpad/+git/launchpad/+merge/446773
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~pelpsi/launchpad:remove-f-string into launchpad:master.

diff --git a/lib/lp/code/model/cibuild.py b/lib/lp/code/model/cibuild.py
index fb29c2e..2ee9c85 100644
--- a/lib/lp/code/model/cibuild.py
+++ b/lib/lp/code/model/cibuild.py
@@ -107,9 +107,10 @@ def get_stages(configuration):
                             or previous_job not in stages[arch][-1][0]
                         ):
                             raise CannotBuild(
-                                f"Job {job_name} would run on {arch},"
-                                + f"but the previous job {previous_job}"
-                                + "in the same pipeline would not"
+                                "Job %r would run on %r,"
+                                + "but the previous job %r"
+                                + "in the same pipeline would not" % job_name,
+                                arch,
                             )
                     jobs[arch].append((job_name, i))