graphprotocol / graphprotocol/graph-node
Synchronize AssemblyScript versions between packages
まだ誰も着手していません。
- 主要言語
- Rust
- スター
- 3.2k
- フォーク
- 1.1k
- 平均マージ
- 4日 1時間
- マージ済み PR(30日)
- 1
説明
Description
Environment
@graphprotocol/graph-cliv0.97.1, which usesassemblyscriptv0.19.23@graphprotocol/graph-tsv0.38.1, which usesassemblyscriptv0.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.
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
@graphprotocol/graph-cli と @graphprotocol/graph-ts の AssemblyScript 依存関係の宣言を比較し、まず各パッケージのパッケージメタデータとビルド設定を確認します。必要な AssemblyScript のアップグレードが両方のパッケージと互換性があることを確認し、null チェックと文字列比較が正常にビルドされることを検証します。両方のパッケージが同期されたバージョンを使用していれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 領域
- build-system, tooling
- issue の種類
- 機能追加
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 静か
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 48/100