martinthomson / i-d-template

A template for IETF internet draft git repositories
Other
204 stars 180 forks source link

Changes for good index (and other markdown) rendering on GitLab and Codeberg #401

Open chrysn opened 12 months ago

chrysn commented 12 months ago

Following up on #399 (which to my knowledge I can't indicate to GitHub in a way that it only shows the commits after "Support non-GitHub hosts for .note.xml generation") [edit: A force push of a fixed commit message after 399 was merged did resolve this], these are changes that'd make using Codeberg & co easier, but they'll probably need some rework after discussion.

As before with #399, I'd like to leave this sit for some time until I request review (while still having it available for the curious).

Prominent points I'm not too happy with are:

  1. The changed setup-readme.sh more or less copies over (and extends) the big per-host switch that is already in build-index.sh. I suppose having a script like wg-meta.sh would be the place to outsource this? [edit: Now doing something very similar to wg-meta.sh in host-meta.sh]
  2. Existing output is changed where GitHub is replaced with github.com. If 1. is properly resolved, there could be a host label that'd be used in its place. [edit: There is now a hostlabel variable generated for known hosters]
chrysn commented 12 months ago

I think I've addressed my own concerns by 1. taking the pattern already established in wg-meta.sh (no need for a function call here, the variables are populated at sourcing time), and 2. applying this to get the old "GitHub" text as a hostlabel that is looked up based on the git host.

The only remaining differences a make -B -f lib/setup.mk now produces to my working repo are

chrysn commented 12 months ago

I think that this is now similarly mature as #399 was: It should not alter the output of GitHub workflows any more.