Skip to content

fix: Pin Python 3.11 in XSTools Windows 2025 build - #4232

Merged
ya4ept merged 1 commit into
masterfrom
bugfix/pin-python-in-workflow
Oct 20, 2026
Merged

ya4ept merged 1 commit into
masterfrom
bugfix/pin-python-in-workflow

Conversation

@alisonrag

Copy link
Copy Markdown
Contributor

This pins Python to 3.11 in GitHub Actions for the windows-2025 runner, ensuring load_module remains available for the bundled scons-local-3.1.2.

The matrix comment was also updated to match, keeping the documented setup consistent with the actual workflow configuration.

This pins Python to 3.11 in GitHub Actions for the windows-2025 runner, ensuring load_module remains available for the bundled scons-local-3.1.2.

The matrix comment was also updated to match, keeping the documented setup consistent with the actual workflow configuration.
@alisonrag

Copy link
Copy Markdown
Contributor Author

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The workflow and its documentation are consistently updated with the intended compatible Python version.

0 open findings

What changed in this PR

Pins Python 3.11 for the Windows 2025 XSTools build to preserve compatibility with bundled SCons.

Changes:

  • Pins the Windows 2025 matrix entry to Python 3.11.
  • Updates the matrix documentation accordingly.
File Description
.github/​workflows/​build_XSTools.yml Pins Python 3.11 and updates the matching workflow comment.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@ya4ept
ya4ept merged commit 6c43b7f into master Oct 20, 2026
17 of 18 checks passed
@ya4ept
ya4ept deleted the bugfix/pin-python-in-workflow branch October 20, 2026 22:26
@ya4ept
ya4ept restored the bugfix/pin-python-in-workflow branch October 20, 2026 22:31
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.

3 participants