Improve consistency of equality between objects

オープン
#279 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
5/5
見積もり時間
1週間以上
初心者へのやさしさ
35/100
issue の種類
リファクタリング
明瞭さ
説明が足りない
活発さ
停滞
技術スタック
javascript
領域
backend

調査の方向性

ファイルやテストの名前は示されていません。まず、issue で説明されている等価性のエントリポイント—equals メソッド、equals ヘルパー、Immutable.is—を確認し、PR #278 の議論を読んでください。完了とは、これらの操作が列挙されたデータ型全体で一貫した可換な結果を生成し、それらの組み合わせがカバーされていることを意味します。

索引モデルが issue の本文から書いたものです。

説明

In the current implementation, equality of data types (e.g., Quantity, Date, DateTime, etc) can be determined in multiple ways:

  1. a.equals(b)
  2. b.equals(a)
  3. equals(a, b) (and perhaps also equals(b, a))

When PR #278 is merged, it will introduce another way:

  1. Immutable.is(a, b)

The results of these operations may differ depending on which one you call and which data types are passed in. This should not be the case, as all equality operations should be consistent and commutative. Ideally, they should all be backed by the same implementation.

For additional discussion, see: https://github.com/cqframework/cql-execution/pull/278#discussion_r988040642.

主要言語
JavaScript
スター
85
フォーク
40
平均マージ
5日 4時間
マージ済み PR(30日)
3

環境構築

  • Dockerfile・Docker Compose ファイルなし
  • プルリクエストのテンプレートあり
  • コントリビューションガイドなし

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

cqframework/cql-execution のほかの issue

cqframework/cql-execution の issue をすべて見る

似ている issue

JavaScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。