graphprotocol / graphprotocol/graph-node

Synchronize AssemblyScript versions between packages

Ouverte
#6,044 7 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

area/runtime area/ts enhancement
Langage dominant
Rust
Étoiles
3.2k
Forks
1.1k
Merge moyen
4 j 1 h
PR mergées (30 j)
1

Description

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.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Comparez les déclarations de dépendances AssemblyScript pour @graphprotocol/graph-cli et @graphprotocol/graph-ts, en commençant par les métadonnées du package et la configuration de build de chaque package. Confirmez que la mise à niveau requise d'AssemblyScript est compatible avec les deux packages et validez que les vérifications de null et les comparaisons de chaînes se compilent correctement ; c'est terminé lorsque les deux packages utilisent la version synchronisée.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Domaine
build-system, tooling
Type d'issue
Fonctionnalité
Difficulté
4/5
Temps estimé
3-5 jours
Activité
Calme
Clarté
Plutôt claire
Accessibilité débutants
48/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.