DiamondLightSource / DiamondLightSource/tickit

Make adapter.after_update optionally async

Open
#169 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
7
Forks
1
PR merge metrics
No merged PRs in 30d

Description

Allow and mange both signatures
```python
def after_update(self) -> None:
...

async def after_update(self) -> None:
...
```

Contributor guide

Open the contributing guide

Research direction

Locate the adapter.after_update hook and inspect how update callbacks are invoked. Trace the existing synchronous path, then verify that both regular and async after_update implementations are supported and covered by the relevant tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.