search

iterate / mikrobloggeriet

0 stars 0 forks source link

Flytt dokument opprettet-logikk til doc-meta #82

Closed teodorlu closed 5 months ago

teodorlu commented 5 months ago

Hva er dette?

Denne endringen er et type grep jeg veldig ofte gjør for å gjøre Clojure-kode lettere å jobbe med: jeg jobber for å få navn til å gi mening i navnerommet sitt. Jeg syntes ikke mikrobloggeriet.serve/read-created-date passet helt inn, hva slags enititet snakket vi om her? Navnerommene mikrobloggeriet.doc og mikrobloggeriet.doc-meta er ment til å snakke om dokumenter og metadata for dokumenter.

Hvorfor review?

Jeg har satt mange på reviewers her, det er fordi jeg synes denne endringen er interessant når man jobber med Clojure.

@JohanMartinEJohnsen og @jraregris: dere jobbet med RSS-logikken orginalt.

Hører gjerne hva dere synes før vi prodsetter!

Er denne endringen trygg?

Ja, jeg er komfortabel med å gå prod nå. Jeg har sjekket at ny og gammel kode gir samme svar med REPL.

Dog: hvis vi hadde hatt noen form for tester for RSS-logikken, hadde den vært tryggere å endre.

teodorlu commented 5 months ago

Clojure-bibliotekene jeg liker å bruke har ofte veldig sterke begreper om navnerom. Hvillke navn passer i hvilke navnerom? Ett navnerom er orientert om ett konsept (ofte en entitet). Så kan man se gjennom navnerommet for å se hva man kan gjøre med entiteten.

teodorlu commented 5 months ago

Går i prod, jeg!