bem / bem/bem-react

@bem-react/di: Check type when setting values in registry

Open
#386 2 comments 3 reactions 0 assignees View on GitHub
improvement
Dominant language
TypeScript
Stars
450
Forks
69
PR merge metrics
No merged PRs in 30d

Description

Неплохо было бы добавить проверку типов при установлении значений в реестре:

```ts
interface RegistryComponents {
ComponentName: ComponentType
}

const registry = new Registry({ id: 'registry-id' })

// Component должен быть типом ComponentType
registry.set('component-key', Component)
```

Но могут возникнуть проблемы при переопределении уже существующего реестра:

```ts
// Скорее всего ругнется, т.к. компонент EnhancedComponent будет иметь другой тип
registry.set('component-key', EnhancedComponent)
```

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.