Skip to content

signals#181

Merged
jerch merged 7 commits intomasterfrom
signals
Jul 10, 2025
Merged

signals#181
jerch merged 7 commits intomasterfrom
signals

Conversation

@jerch
Copy link
Collaborator

@jerch jerch commented Jul 10, 2025

Second approach to a signal implementation, this time much more light-weight with only 3 simple signals:

  • resolver_start to indicate the start of a resolver update
  • resolver_update sent immediately after a model bulk update
  • resolver_exit to indicate the exit of a resolver update

This lighter implementation exposes the resolver_update signals directly from the ongoing dependency tree update, which has several downsides, like being under a transaction and the state not being fully in sync again. --> Needs proper documentation of the shortcomings.

TODO:

Fixes #56 finally.

@jerch jerch added this to the 0.3.0 milestone Jul 10, 2025
@coveralls
Copy link

coveralls commented Jul 10, 2025

Pull Request Test Coverage Report for Build 16204997570

Details

  • 22 of 22 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.1%) to 95.221%

Totals Coverage Status
Change from base Build 16201466741: 0.1%
Covered Lines: 1169
Relevant Lines: 1215

💛 - Coveralls

@jerch jerch mentioned this pull request Jul 10, 2025
@jerch jerch merged commit 1558d30 into master Jul 10, 2025
15 checks passed
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.

notifications from resolver CF updates

2 participants