Tensegritics/ClojureDart

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

Ouverte

#218 ouverte le 3 avr. 2023

Voir sur GitHub
 (1 commentaire) (0 réaction) (0 personne assignée)Clojure (115 forks)batch import
enhancementgood first issue

Métriques du dépôt

Stars
 (1 618 étoiles)
Métriques de merge PR
 (Merge moyen 8j) (2 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Inspectez l'implémentation de la macro defprotocol dans la base de code de ClojureDart pour ajouter la prise en charge de l'option :extend via metadata. Suivez la manière dont les options de protocole sont analysées et assurez vous que :extend via metadata est correctement géré.
Stack technique
Aucun
Domaine
tooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
ClojureGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60