search

pytest-dev / pluggy

A minimalist production ready plugin system
https://pluggy.readthedocs.io/en/latest/
MIT License
1.23k stars 121 forks source link

hooks: fix plugins registering other plugins in a hook #473

Closed bluetech closed 7 months ago

bluetech commented 7 months ago

when the other plugins implement the same hook itself.

Fix #438. Regressed in 63b7e908b4b22c30d86cd2cff240b3b7aa6da596 - pluggy 1.1.0.

Went with the simple solution described in the issue for now, basically undoing most of the rationale for 63b7e908b4b22c30d86cd2cf (though I think it's still better to have a single _hookimpls list).

bluetech commented 7 months ago

Should we add a CHANGELOG entry too?

I will update the changelogs before doing a release. I will do a release in the next few days.