-
Notifications
You must be signed in to change notification settings - Fork 5
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
[IMP] manifest: add category property #57
Conversation
src/manifestoo_core/addon.py
Outdated
@@ -53,6 +53,7 @@ def from_addon_dir( | |||
cls, | |||
addon_dir: Path, | |||
allow_not_installable: bool = False, | |||
addons_categories: Optional[List[str]] = None, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this kind of filtering belongs here, because we can imagine an infinity of different such filters.
Let's keep that in manifestoo for now.
@@ -128,6 +128,10 @@ def license(self) -> Optional[str]: | |||
def author(self) -> Optional[str]: | |||
return self._get("author", _check_optional_str, default=None) | |||
|
|||
@property | |||
def category(self) -> Optional[str]: | |||
return self._get("category", _check_optional_str, default=None) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
788575c
to
27ad10f
Compare
Done |
@@ -136,3 +136,6 @@ dmypy.json | |||
|
|||
# Cython debug symbols | |||
cython_debug/ | |||
|
|||
# pycharm dev env | |||
.idea/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tip: add this to your ~/.gitconfig
[core]
excludesfile = ~/.gitignore
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! I'll do a release soon.
27ad10f
to
8883e82
Compare
8883e82
to
21e8f79
Compare
Add the category property in manifest.