search

iterate / mikrobloggeriet

0 stars 0 forks source link

rel="self"-link bør linke til feeden, ikke bloggen #38

Closed jraregris closed 8 months ago

jraregris commented 9 months ago

Om man validerer RSS-feeden med validator.w3.org blir man anbefalt å endre...

<atom:link href="https://mikrobloggeriet.no" rel="self" type="application/rss+xml"></atom:link>

...til å linke til selve feeden, og ikke hoved-url-en til nettstedet. (https://validator.w3.org/feed/docs/warning/SelfDoesntMatchLocation.html)

Heldigvis har vi allerede støtte for det i RSS-koden, om vi bare sender med :feed-url, slik jeg gjør i denne PR-en.

teodorlu commented 9 months ago

Da er det bare å trykke "squash and merge" her, @jraregris!

Normen har vært at den som skriver koden merger selv.

jraregris commented 9 months ago

Da er det bare å trykke "squash and merge" her, @jraregris!

Hvorfor skal jeg squashe én commit?

teodorlu commented 9 months ago

Du kan merge direkte også.

Jeg har likt å få én commit per PR på master. Da opplever jeg at endringene er mer atomiske. én commit, én problembeskrivelse, én løsning.

Med én commit blir som du påpeker resultatet det samme om man merger eller squash-merger.

teodorlu commented 9 months ago

Når det er sagt, det er opp til @JohanMartinEJohnsen og @olavm hvordan de ønsker å jobbe med Mikrobloggeriet framover.

Men jeg gjetter at det er OK for dem å fortsette med PR-er som vi har gjort før.