Skip to content

Conversation

VojtechStep
Copy link
Collaborator

@VojtechStep VojtechStep commented Aug 6, 2025

  • Updated the list of contributors in citations
    • This amounts to adding myself to CITATION.cff; hopefully shouldn't be controversial, as I discussed it with Egbert over a year ago, but never pushed it
  • Added a pre-commit check for rejecting links in headings. Since mdbook turns headings into links with no checks on the text, this leads to nested links, which are not supported by HTML. Browsers would recover by interpreting <a>Before <a>inside</a> after</a> as <a>Before </a><a>inside</a> after
    • Correspondingly I fixed all the occurrences of those headings. Two of them were linked to from the BdJR24 paper, but the ids stayed the same, so the links are still valid
  • Fixed The module index generator doesn't support modules with unicode characters in their names  #1027 by both not failing silently on unicode file names (they are treated correctly now), and rejecting files with non-ASCII names in a pre-commit hook
  • Fixed Run link-check sooner, and irrespective of other pipelines #775 by implementing the "source filename"->"Agda processed filename" translation in Python and using this instead of Agda when asked
    • Updated CI accordingly, so that linkcheck should run in parallel to typechecking, with no typechecking required
  • Fixed Concept macro incompatible with inline latex? #1362 by explicitly rejecting LaTeX fragments in concepts
  • Typos

Copy link
Collaborator

@fredrik-bakke fredrik-bakke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thanks for looking into these old issues!

@VojtechStep VojtechStep force-pushed the feature/website-corrections branch from ddd8203 to 62e7ec7 Compare August 7, 2025 14:52
@VojtechStep
Copy link
Collaborator Author

I stuffed another feature in the last two commits, namely "fixing" the concepts/latex incompatibility by making it explicitly rejected

@fredrik-bakke
Copy link
Collaborator

Looks good to me! Feel free to merge when you are ready

@VojtechStep VojtechStep merged commit 2d7742f into UniMath:master Aug 9, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants