graphprotocol / graphprotocol/graph-node

Synchronize AssemblyScript versions between packages

Abierto
#6,044 7 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

area/runtime area/ts enhancement
Lenguaje dominante
Rust
Estrellas
3.2k
Forks
1.1k
Merge medio
4 d 1 h
PR fusionados (30 d)
1

Descripción

Description

Environment

  • @graphprotocol/graph-cli v0.97.1, which uses assemblyscript v0.19.23
  • @graphprotocol/graph-ts v0.38.1, which uses assemblyscript v0.27.31

Problem

keywords: assemblyscript, equality, string checks, string comparison, null check, ==, ===, !=, !==

In previous versions of graph-ts, we were able to perform null checks using ==. But now, in the latest versions, we are forced to use === (otherwise, we get all sorts of errors at build time).

I wouldn't have a problem with using ===, but we are also forced to use == for string comparisons.

This is a super error-prone design where the maintainers have to remember to use slightly different operators for different types.

Solution

Upgrade to AssemblyScript v0.20, in which the === operator no longer checks for strict object equality.

https://github.com/AssemblyScript/assemblyscript/issues/621

Are you aware of any blockers that must be resolved before implementing this feature? If so, which? Link to any relevant GitHub issues.

No response

Some information to help us out
  • Tick this box if you plan on implementing this feature yourself.
  • I have searched the issue tracker to make sure this issue is not a duplicate.

Guía de contribución

Abrir la guía de contribución

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.

Línea de trabajo

Compara las declaraciones de dependencias de AssemblyScript para @graphprotocol/graph-cli y @graphprotocol/graph-ts, comenzando por los metadatos del paquete y la configuración de build de cada paquete. Confirma que la actualización requerida de AssemblyScript es compatible con ambos paquetes y valida que las comprobaciones de null y las comparaciones de strings se compilen correctamente; se considera terminado cuando ambos paquetes usan la versión sincronizada.

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

Evaluación

Área
build-system, tooling
Tipo de issue
Nueva funcionalidad
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Tranquilo
Claridad
Bastante claro
Aptitud para principiantes
48/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.