Tensegritics/ClojureDart

Add support for :extend-via-metadata in protocol definitions

Aperta

#218 aperta il 3 apr 2023

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Clojure (115 fork)batch import
enhancementgood first issue

Metriche repository

Star
 (1618 stelle)
Metriche merge PR
 (Merge medio 8g) (2 PR mergiate in 30 g)

Descrizione

Describe the bug Code can not be compiled once defprotocol elect to be extended via per-value metadata.

To Reproduce Steps to reproduce the behavior:

  1. Create a CLJD project
  2. Try to compile this code:
(defprotocol  MyProtocol :extend-via-metadata true
              (some-fn [this]))
  1. See error
Don't know how to create ISeq from: clojure.lang.Keyword
Faulty form (defprotocol MyProtocol :extend-via-metadata true (some-fn [this]))

Expected behavior

  • code can be compiled and metadata + protocols can be used.

Additional context

  • I have experienced this error when trying to use Honey.SQL in CLJD

Guida contributor

Direzione di ricerca
Ispeziona l'implementazione della macro defprotocol nel codebase di ClojureDart per aggiungere il supporto per l'opzione :extend via metadata. Traccia come vengono analizzate le opzioni del protocollo e assicurati che :extend via metadata sia gestito correttamente.
Tech stack
Nessuno
Dominio
tooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ClojureGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60