Improve support for `exactOptionalPropertyTypes`
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 68/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- backend-api-design
Línea de trabajo
Empieza por localizar la interfaz Options<P, C> que aparece en el issue y comprueba cómo están declaradas y se usan sus propiedades opcionales. Actualiza los tipos de las propiedades opcionales para permitir undefined explícito y, después, ejecuta las comprobaciones o pruebas de TypeScript pertinentes para confirmar que los tipos aceptan esas asignaciones con exactOptionalPropertyTypes.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The Options type is defined as:
export interface Options<P, C> {
/** Minimum zoom level at which clusters are generated. @default 0 */
minZoom?: number;
/** Maximum zoom level at which clusters are generated; capped at 30. @default 16 */
maxZoom?: number;
/** Minimum number of points to form a cluster. @default 2 */
minPoints?: number;
/** Cluster radius, in pixels. @default 40 */
radius?: number;
/** (Tiles) Tile extent; radius is calculated relative to this value. @default 512 */
extent?: number;
/** Size of the KD-tree leaf node; affects performance. @default 64 */
nodeSize?: number;
/** Whether timing info should be logged. @default false */
log?: boolean;
/** Whether to generate ids for input features in vector tiles. @default false */
generateId?: boolean;
/** Cluster properties for a single point, e.g. `props => ({sum: props.myValue})`. Defaults to identity. */
map?: (props: P) => C;
/** Merges two clusters' properties, e.g. `(accumulated, props) => { accumulated.sum += props.sum; }` */
reduce?: ((accumulated: C, props: Readonly<C>) => void) | null;
}
If I enable exactOptionalPropertyTypes to true, I am no longer able to set map and other properties to undefined. Please consider adding undefined to the optional properties so they can be explicitly set to undefined as well.
- Lenguaje dominante
- JavaScript
- Estrellas
- 2.4k
- Forks
- 303
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de mapbox/supercluster
-
bug
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
mapbox/supercluster#254 · 2 comentarios ·
-
Missing points when bounds are scoped to the extent of pointsQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abiertoquestion
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
mapbox/supercluster#234 · 1 comentario ·
-
Caching SuperCluster IndexAbiertoenhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
mapbox/supercluster#218 · 4 comentarios ·
-
Nested geoJson properties are copied by referencePosiblemente ocupada @mrkwanzaa la tomó hace 1339 días. Abiertoenhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 50/100
mapbox/supercluster#215 ·
-
enhancement
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
mapbox/supercluster#211 · 2 comentarios ·
Todos los issues de mapbox/supercluster
Issues similares
-
Pills' meter dots fade on bright and deep pillsPosiblemente ocupada @wassim-azi la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
wassim-azi/agy-powerline#168 ·
Los mantenedores suelen responder en 1 día
-
accounts bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día
-
area:docs effort:S impact:S kind:docs source:critic stage:ready
Dificultad 1/5 1-3 horas Aptitud para principiantes 91/100
Jason-Vaughan/TangleClaw#2693 ·
Los mantenedores suelen responder en 1 día
-
infrastructure P2 security
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Energy flow: add an "Other" row so the consumers add up to the consumption totalPosiblemente ocupada @github-actions la tomó hoy. Abiertoenhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
evcc-io/evcc#35512 · 1 comentario ·
Los mantenedores suelen responder en 1 día