Skip to content
This repository was archived by the owner on Aug 11, 2025. It is now read-only.
This repository was archived by the owner on Aug 11, 2025. It is now read-only.

Remove need to explicitly make injectables async compatible #20

@HampusMat

Description

@HampusMat

The need to explicitly make injectables async compatible should be removed.

This would be simple if not for the fact that having the async crate feature enable this by default would make it have a non-additive change. More about that here.

An alternative approach to this would be to only implement AsyncInjectable when the type, dependencies and interface are Send + Sync + 'static. This however is not possible (as far as i know) as long as trivial constraints is not stabilized. And even if it becomes stabilized, using it would break the 1.62.1 MSRV.

More research needs to be done about this

Metadata

Metadata

Assignees

No one assigned

    Labels

    featureA feature requesthelp wantedExtra attention is needed

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions