Adopt InternMap for selection.data

Abierto
  • ¿Algo no está bien?
#291 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
45/100
Tipo de issue
Nueva funcionalidad
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
d3, javascript
Área
frontend

Línea de trabajo

Read src/selection/data.js at the linked line and check how the current string coercion is used in the data join. The issue proposes InternMap for consistency; inspect how the project uses InternMap and identify the relevant tests. Done means the join uses InternMap as intended and the tests cover the resulting behavior.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Currently we coerce to a string, but we should use InternMap for consistency:

https://github.com/d3/d3-selection/blob/91245ee124ec4dd491e498ecbdc9679d75332b49/src/selection/data.js#L44

Lenguaje dominante
JavaScript
Estrellas
571
Forks
283
Merge medio
7 h
PR fusionados (30 d)
1

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de d3/d3-selection

Todos los issues de d3/d3-selection

Issues similares

Más issues de JavaScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.