Tensegritics/ClojureDart

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

Offen

#218 geöffnet am 03.04.2023

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Clojure (115 Forks)batch import
enhancementgood first issue

Repository-Metriken

Stars
 (1.618 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (2 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuche die Implementierung des defprotocol Makros im ClojureDart Codebestand, um Unterstützung für die Option :extend via metadata hinzuzufügen. Verfolge, wie Protokolloptionen geparst werden, und stelle sicher, dass :extend via metadata korrekt behandelt wird.
Tech Stack
Keine
Domain
tooling
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
ClojureGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60