Merge ~adam-collard/maas/+git/maas-release-tools:cached-me into ~maas-committers/maas/+git/maas-release-tools:main

Proposed by Adam Collard
Status: Merged
Merged at revision: 8d12e0480e7d8d0922fce3b99bb3ae3242a8ee2e
Proposed branch: ~adam-collard/maas/+git/maas-release-tools:cached-me
Merge into: ~maas-committers/maas/+git/maas-release-tools:main
Diff against target: 21 lines (+2/-1)
1 file modified
maas_release_tools/launchpad.py (+2/-1)
Reviewer Review Type Date Requested Status
Christian Grabowski Approve
Review via email: mp+422838@code.launchpad.net

Commit message

Cache me!

To post a comment you must log in.
Revision history for this message
Christian Grabowski (cgrabowski) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/maas_release_tools/launchpad.py b/maas_release_tools/launchpad.py
2index 25d7db7..1b73405 100644
3--- a/maas_release_tools/launchpad.py
4+++ b/maas_release_tools/launchpad.py
5@@ -2,6 +2,7 @@
6
7 from contextlib import contextmanager
8 from datetime import datetime
9+from functools import cached_property
10 import logging
11 from pathlib import Path
12 from typing import Optional, Sequence
13@@ -36,7 +37,7 @@ class LaunchpadActions:
14 if not self._project:
15 raise UnknownLaunchpadEntry("project", project)
16
17- @property
18+ @cached_property
19 def me(self):
20 """Return the logged in user user from LP."""
21 return self.lp.me

Subscribers

People subscribed via source and target branches

to all changes: