Skip to content

Fix Extensions Index CMakeLists.txt checks - #2

Merged
MarkDanielArndt merged 2 commits into
MarkDanielArndt:mainfrom
JonaRichter:main
Jul 9, 2026
Merged

Fix Extensions Index CMakeLists.txt checks#2
MarkDanielArndt merged 2 commits into
MarkDanielArndt:mainfrom
JonaRichter:main

Conversation

@JonaRichter

@JonaRichter JonaRichter commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Fix EXTENSION_SCREENSHOTURLS format: the Slicer Extensions Index checker requires the set() call on a single line with no linebreak before the closing quote/paren (set(EXTENSION_SCREENSHOTURLS "...")). The previous multi-line format (both as 4 separate quoted URLs and as a single wrapped string) failed this check with "No EXTENSION_SCREENSHOTURLS found in CMakeLists.txt."
  • Update EXTENSION_ICONURL and all EXTENSION_SCREENSHOTURLS to point to the main branch instead of the no-longer-existing master branch, so the referenced images actually resolve.
  • Remove the dead master branch trigger entries in .github/workflows/ci.yml (main was already covered; master no longer exists).

Test plan

  • Confirm Extensions Index automated check passes on the referenced CMakeLists.txt content

@MarkDanielArndt
MarkDanielArndt merged commit 2088f83 into MarkDanielArndt:main Jul 9, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants