-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move LXD and LXDVM to checkbox-support (New) #1645
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1645 +/- ##
==========================================
+ Coverage 48.85% 48.88% +0.02%
==========================================
Files 370 372 +2
Lines 40234 40245 +11
Branches 6794 6793 -1
==========================================
+ Hits 19657 19674 +17
+ Misses 19857 19850 -7
- Partials 720 721 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Mr. @Hook25 It seems the |
feel free to duplicate the implementation, we don't want checkbox-support to depend on plainbox. Also consider using the builtin one try:
from functools import cached_property
except ImportError:
class cached_property:
"""
Decorator that converts a method with a single self argument into a
property cached on the instance.
See https://goo.gl/QgJYks (django cached_property)
"""
def __init__(self, func):
self.func = func
def __get__(self, instance, type=None):
if instance is None:
return self
res = instance.__dict__[self.func.__name__] = self.func(instance)
return res |
Try to import from `functools` first.
Description
Moved
LXD
andLXDVM
(introduced by #1577) tocheckbox-support
.Resolved issues
Resolves CHECKBOX-1696
Documentation
Tests
checkbox-support
.