Skip to content

[FIX] runtests: Ignore base_automation from dependencies#359

Open
grindtildeath wants to merge 1 commit intocamptocamp:masterfrom
grindtildeath:fix-ignore_base_automation
Open

[FIX] runtests: Ignore base_automation from dependencies#359
grindtildeath wants to merge 1 commit intocamptocamp:masterfrom
grindtildeath:fix-ignore_base_automation

Conversation

@grindtildeath
Copy link
Member

When base_automation appears in the dependencies, the register hook from this module will be executed before any local module will be installed, potentially messing with MRO of models overriden in local modules that are not installed yet.

gurneyalex
gurneyalex previously approved these changes Oct 31, 2025
@gurneyalex gurneyalex self-requested a review October 31, 2025 07:35
Copy link
Member

@gurneyalex gurneyalex left a comment

Choose a reason for hiding this comment

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

can you check the CI?

@gurneyalex gurneyalex dismissed their stale review October 31, 2025 07:36

CI is red

When base_automation appears in the dependencies, the register hook from
this module will be executed before any local module will be installed,
potentially messing with MRO of models overriden in local modules
that are not installed yet.
@grindtildeath grindtildeath force-pushed the fix-ignore_base_automation branch from 968aa3e to 5c798b9 Compare February 26, 2026 13:52
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.

3 participants