Skip to content

Conversation

@muratmaga
Copy link
Contributor

This should fix the stable versions of these two extensions, Photogrammetry and ANTsPy, to branch "stable-5.10". Preview versions should continue to use the master/main. The other PR can be closed without merging (as it was trying to point out the development version to a different branch).

This is the one that needs to be merged.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR pins stable versions for several Slicer extensions, primarily focusing on ANTsPy and Photogrammetry as mentioned in the description, while also updating SlicerMorph, QuantitativeReporting, and LanguagePacks configurations.

Key Changes:

  • Pin ANTsPy, Photogrammetry, and SlicerMorph extensions to the stable-5.10 branch
  • Pin QuantitativeReporting to a specific commit hash
  • Update LanguagePacks repository URL to point to SoniaPujolLab organization

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
SlicerANTsPy.json Updated scm_revision from "main" to "stable-5.10" branch
Photogrammetry.json Updated scm_revision from "master" to "stable-5.10" branch
SlicerMorph.json Updated scm_revision from "master" to "stable-5.10" branch
QuantitativeReporting.json Updated scm_revision from "master" to specific commit hash 2c0112785c9ba3af9be6a1f86d5ed7bb8b700705
LanguagePacks.json Updated scm_url from Slicer/SlicerLanguagePacks to SoniaPujolLab/SlicerLanguagePacks

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@pieper
Copy link
Member

pieper commented Dec 7, 2025

Looks like you have LanguagePacks and QuantitativeReporting in this PR too, but I suppose you didn't mean to.

@muratmaga
Copy link
Contributor Author

No, not sure where does come from?

@muratmaga muratmaga closed this Dec 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants