Improve consistency of equality between objects
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 35/100
- Tipo di issue
- Refactoring
- Chiarezza
- Da chiarire
- Stato di attività
- Ferma
- Stack tecnologico
- javascript
- Ambito
- backend
Direzione di ricerca
Non vengono indicati file o test. Inizia esaminando i punti di ingresso dell’uguaglianza descritti nell’issue—i metodi equals, l’helper equals e Immutable.is—e leggi la discussione sulla PR #278. Il lavoro sarà considerato completato quando queste operazioni produrranno risultati coerenti e commutativi tra i tipi di dati elencati, con copertura delle loro combinazioni.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
In the current implementation, equality of data types (e.g., Quantity, Date, DateTime, etc) can be determined in multiple ways:
a.equals(b)b.equals(a)equals(a, b)(and perhaps alsoequals(b, a))
When PR #278 is merged, it will introduce another way:
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.
- Lingua principale
- JavaScript
- Stelle
- 84
- Fork
- 41
- Merge medio
- 5g 4h
- PR unite (30g)
- 3
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Nessuna guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di cqframework/cql-execution
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 72/100
cqframework/cql-execution#400 · 1 commento ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 50/100
cqframework/cql-execution#399 · 3 commenti ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 72/100
cqframework/cql-execution#398 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
cqframework/cql-execution#397 ·
-
Issue with using type testing operation on FHIR primitivesForse già presa @dehall l’ha presa 15 giorni fa. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 54/100
cqframework/cql-execution#391 · 2 commenti · 1 assegnatario ·
Tutte le issue di cqframework/cql-execution
Issue simili
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
arfct/timezone.fyi#19 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
fastify/fastify-http-proxy#493 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
tensorflow/tfjs#8754 ·
-
Marketplace time-zone issueApertabug main ui
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
openhab/openhab-webui#4671 ·
I maintainer di solito rispondono entro 1 giorno
-
scratch-paint's npm test does not run lint, so CI never lints paint; one lint error is currently in developForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
scratchfoundation/scratch-editor#754 ·
I maintainer di solito rispondono entro 1 giorno