Skip to content

#351 - Type the task registry on TaskRegistryInterface - #352

Open
njoubert-cleverage wants to merge 1 commit into
mainfrom
351
Open

njoubert-cleverage wants to merge 1 commit into
mainfrom
351

Conversation

@njoubert-cleverage

Copy link
Copy Markdown
Member

Description

Fixes #351.

  • ProcessManager and ProcessHelpCommand: constructor argument and property typed on TaskRegistryInterface instead of the final TaskRegistry (same as ui-process-bundle#186). The deprecation messages are unchanged.
  • New test with a custom TaskRegistryInterface implementation.

Requirements

  • Documentation updates
    • Reference
    • Changelog
  • Unit tests

Breaking changes

None for the callers (wider type). A subclass redeclaring the protected $taskRegistry property with the TaskRegistry type would have to widen it.

🤖 Generated with Claude Code

…on TaskRegistryInterface. Add tests.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
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.

ProcessManager and ProcessHelpCommand: type the task registry on TaskRegistryInterface

1 participant