andrewmcodes-archive / andrewmcodes-archive/tailwind_view_component

refactor: Classify must be rearchitected

Offen
#1 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Ruby
Sterne
5
Forks
0
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Classify.rb is a pattern I borrowed from the Primer team but it's just not going to work for us. I've been really thinking about the best way to go about this to avoid duplication and be flexible and I believe the answer is sourcing class and constraints directly from the Tailwind config.

This presents some issues though bc the config is a JS file. I wrote a script to convert it to JSON but it's brittle and I don't love it.

I'm wondering if this would be a good use case for a JSON schema that we generate for each TW release and combine it with some of the `dry-rb` tools.

Open to suggestions but my goal is to separate (at minimum) the class config stuff into separate files based on their category on the Tailwind docs site.

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.