Skip to content

feat: allow excluding addons by author#85

Open
yajo wants to merge 2 commits intoacsone:mainfrom
moduon:exclude-authors
Open

feat: allow excluding addons by author#85
yajo wants to merge 2 commits intoacsone:mainfrom
moduon:exclude-authors

Conversation

@yajo
Copy link
Contributor

@yajo yajo commented Oct 31, 2025

Authors such as Odoo S.A. or Odoo Community Association have good CI policies, so you might want to avoid testing their addons on your own infrastructure.

With this new feature, you can find all addons except those belonging to certain authors, providing an easy way to achieve that goal.

@moduon MT-1075

@yajo yajo force-pushed the exclude-authors branch 5 times, most recently from 1eb6788 to 6fe2756 Compare October 31, 2025 11:36
Authors such as Odoo S.A. or Odoo Community Association have good CI
policies, so you might want to avoid testing their addons on your own
infrastructure.

With this new feature, you can find all addons except those belonging to
certain authors, providing an easy way to achieve that goal.

@moduon MT-1075
Copy link
Member

@sbidoul sbidoul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, code looks good.

Of course this does not scale to any kind of selectors users will imagine over time.

Wondering if it would make sense to have include/exclude options accepting a boolean expression on Manifest objects. I guess eval would be acceptable if the only thing in the evaluation context is a Manifest and we document clearly that input to that option must come from trusted sources.

.installed.cfg
*.egg
MANIFEST
uv.lock
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
uv.lock

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this helps if you don't want to commit that file. I just did uv sync to start developing and it got created.

Co-authored-by: Stéphane Bidoul <stephane.bidoul@acsone.eu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants